milleruntime commented on PR #2777: URL: https://github.com/apache/accumulo/pull/2777#issuecomment-1156336596
> Its kinda neat that Java 18 found this. @milleruntime would you happen to have the error/warn message? I am curious what it said. I currently only have Java 18 in my IDE and this is all it is saying: <pre> workspace/accumulo/core/src/main/java/org/apache/accumulo/core/security/Authorizations.java:47:29 java: non-transient instance field of a serializable class declared with a non-serializable type </pre> > Reading that doc it seems like removing it may only break binary compat in the case where code is casting to Serializable. Does that interpretation seem correct? I will take a look at that link, thanks. > Since serialization did not work, maybe no one was doing that and its ok to remove if code that was not casting to Serializable is ok. So the serialization of `Authorizations` _does_ work as it is currently written. See https://github.com/apache/accumulo/pull/2777#issuecomment-1155517364. I am going to create a test for all the Serialization warnings so we can figure out a way forward for https://github.com/apache/accumulo/issues/2776. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
