iamaleksey commented on PR #4774: URL: https://github.com/apache/cassandra/pull/4774#issuecomment-4439447310
The idea is (and was when writing/reviewing the similar patch for Accord) is that we keep all serializers that don't yet need versioning (and might potentially never need it) as unversioned, and only convert them to versioned when serialization changes. This is fine so long as top level serializers (messages, sstable structures, etc.) themselves are versioned. We were also going to audit *all* C* serializers and convert all versioned serializers that don't branch their logic on versions to unversioned, but never got around to it. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
