johnzon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [johnzon] mkarg commented on issue #44: Fix for [JOHNZON-276]: StackOverflowError
Date Mon, 07 Oct 2019 10:24:15 GMT
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

Mime
View raw message