You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@johnzon.apache.org by GitBox <gi...@apache.org> on 2019/10/07 10:24:15 UTC

[GitHub] [johnzon] mkarg commented on issue #44: Fix for [JOHNZON-276]: StackOverflowError

mkarg commented on issue #44: Fix for [JOHNZON-276]: StackOverflowError
URL: https://github.com/apache/johnzon/pull/44#issuecomment-538938554
 
 
   > @mkarg if you can push a github repo with a failling test i can try to have a look
   
   @rmannibucau Again, thanks for this kind offer. In fact, just now I resolved the case on my own (by using latest Johnzon 1.2.1 including all the fixes of the last weeks), plus write an adapter. So I now was able to get rid of the explicit `@JsonVisibility(DefaultPropertyVisibilityStrategy.class)`.
   
   Having said that, I *now* do agree to your original claim: I do **not** see a need for `DefaultPropertyVisibilityStrategy` to be `public` anymore now that 1.2.1 is published. :-)
   
   So the question is, how to go on with this PR / Issue 276?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services