cshannon commented on PR #3833:
URL: https://github.com/apache/accumulo/pull/3833#issuecomment-1758384969

   > Did the serialization of these jsons change? This seems to suggest that 
they are now serialized with the escape characters.
   > 
   > The issue with the failing test case is how the test json is being created 
using `createSelectedFilesJson()`. I can make a PR against your branch to try 
to change this method to use GSON to construct this json to see if it solves 
the issue.
   
   @DomGarguilo - I actually I think you are right, i think this is just a test 
issue. We do want to escape it as it's embedded json and that is fine. The test 
just needs to account for it. I can make the change real fast.


-- 
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]

Reply via email to