milleruntime commented on pull request #1774: URL: https://github.com/apache/accumulo/pull/1774#issuecomment-726199653
I also realized that moving `TServerInstance` will most likely break serialization so we may have to do something in the upgrade code to prevent issues upgrading. This class is weird though... there are `writeObject(out)` and `readObject(in)` methods which look similar to Thrift classes and the class starting with "T" make me think that maybe this was based on a thrift class at one point. I had thought the "T" was for Tablet Server. ---------------------------------------------------------------- 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]
