[PR] Disables HTML escaping in LoadPlan json encoding. [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner opened a new pull request, #5299:
URL: https://github.com/apache/accumulo/pull/5299

   The encoding in LoadPlan was using GSon default settings which encoded some HTML chars like `=` as unicode escape sequences.  This did not cause any problems because GSon would reverse this encoding.  However it was not documented in the javadoc for LoadPlan and it seems unnecessary.
   
   Disabled the HTML escaping so the code aligns w/ the javadoc.  Also updated the unit test to produce `=` in the base64 and see how that behaves in the json.  This allows the test to better detect changes in the way the code persist data.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Disables HTML escaping in LoadPlan json encoding. [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner merged PR #5299:
URL: https://github.com/apache/accumulo/pull/5299


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org