stoty commented on pull request #1053: URL: https://github.com/apache/phoenix/pull/1053#issuecomment-754436022
As we don't seem to use the java serialization framework anywhere, and your patch doesn't change anything WRT the java serialization situation, It doesn't really matter. You may as well make the class Serializable, as it has no state, and it wouldn't make any difference to us, and silence spotbugs, but it's good either way. ---------------------------------------------------------------- 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]
