dlmarion commented on PR #5229: URL: https://github.com/apache/accumulo/pull/5229#issuecomment-2575870084
In #5060 @cshannon said: > I took a look at this and was thinking a bit about it and the only really good way to get rid of the escaping would be to not use the serialized json string of the files and instead use an object representation of StoredTabletFile, but that would break the encapsulation of StoredTabletFile and expose the file/range outside of it so I'm not sure if that's a great idea. I think this is the way. In 4be28a1 I created an object to store the path, start, and end rows. The input files are now an array of these objects and the double-quotes in the output are not escaped. However, they are still escaped for the intermediate output file, so we may want to change that as well. -- 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]
