keith-turner commented on PR #4215: URL: https://github.com/apache/accumulo/pull/4215#issuecomment-1924151773
> The key will be if the serialized forms are stored externally. Currently I believe serialization is used with Fates and if that how this class is stored - that would not present a problem in this case because Fates are not allowed to span Accumulo version changes. I think that is correct, I don't think this change will be a problem. Looking at org.apache.accumulo.manager.tableOps.tableExport.WriteExportFiles that is what writes out data that would go across Accumulo versions. I did not see it using the class changed in this PR. Unrelated to this issue, did notice that WriteExportFiles.exportConfig() could probably be improved to use the newer config code in 2.1 that would allow reading only the props explicitly set for a table. -- 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]
