DomGarguilo opened a new pull request, #6179: URL: https://github.com/apache/accumulo/pull/6179
MAC writes to a `.properties` file in a way that breaks newline JSON values. This happened because `MiniAccumuloClusterImpl.writeConfigProperties()` manually wrote the `key=value` to the file without proper newline escaping. This PR uses Properties.store() for for .properties serialization so we can remove the manual code and properly handle newlines. I did not add any new tests for this but with the newline escaping removed from ScanServerGroupConfigurationIT, the test passes with these changes to `writeConfigProperties()` and throws a json parse exception without them. Note: This PR is against main since I wasn't sure if this was something we wanted to change in 2.1 or not. -- 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]
