reta commented on pull request #73: URL: https://github.com/apache/johnzon/pull/73#issuecomment-766164766
@rmannibucau Mapping a not opened/imported model is certainly not advisable, probably the question would be: should `Johnzon` do best effort by not trying to access what it cannot, or just fail fast. Sadly I do not have the only right answer, but I think certain measures from the library/framework could help. Knowing that the semantic of the `setAccessible` has changed, try to access and skip if it is not possible (log warning may be?). As for the user of the library, it is logical and reasonable expectation, but this is just my opinion. ---------------------------------------------------------------- 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: [email protected]
