Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Objects are not serializable #54

Closed
alkan opened this issue Sep 30, 2013 · 4 comments
Closed

Objects are not serializable #54

alkan opened this issue Sep 30, 2013 · 4 comments
Assignees

Comments

@alkan
Copy link

@alkan alkan commented Sep 30, 2013

Version: 1.0.4.1

I am using OWNER framework with Twitter/Storm. Such frameworks requires all instance variables should be serializable. Could you please make all objects you define be serializable?

@lviggiano
Copy link
Owner

@lviggiano lviggiano commented Sep 30, 2013

👍

I plan to release 1.0.5 this week. This change will be included.

@lviggiano
Copy link
Owner

@lviggiano lviggiano commented Sep 30, 2013

The change you have requested is implemented on master. If you would like to test it on your project you need to package your own jar, see Building from sources.

Once you have done mvn install, you can declare the dependency from org.aeonbits.owner:owner:1.0.5-SNAPSHOT in your pom.xml and test your code against the latest version.

Let me know, if there is still any issue related to this.

@ghost ghost assigned lviggiano Sep 30, 2013
@alkan
Copy link
Author

@alkan alkan commented Oct 1, 2013

Thank you for your prompt respond Luigi.

On 09/30/2013 07:06 PM, Luigi R. Viggiano wrote:

The change you have requested is implemented on master. If you would
like to test it on your project you need to package your own jar, see
Building from sources http://owner.aeonbits.org/docs/building/.

Once you have done |mvn install|, you can declare the dependency from
|org.aeonbits.owner:owner:1.0.5-SNAPSHOT| and test your code against the
latest version.

Let me know, if there is still any issue related to this.


Reply to this email directly or view it on GitHub
#54 (comment).

@lviggiano
Copy link
Owner

@lviggiano lviggiano commented Oct 9, 2013

released today in version 1.0.5. it's available in maven central repository; @alkan you can update your app to have this feature enabled.

@lviggiano lviggiano closed this Oct 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.