ctubbsii opened a new pull request, #3074: URL: https://github.com/apache/accumulo/pull/3074
* Delete ClientConfiguration and unit test * Update ClientConfConverter to remove legacy client config * Update ClientConfConverterTest.basic to test conversion directly to/from client Properties and AccumuloConfiguration (which is still needed for some bootstrapping code and SASL configuration code) * Remove legacy trace props from ClientProperty * Remove deprecated getClientConfig() methods from minicluster * No longer write out client config file (only client properties file) and no longer set corresponding `ACCUMULO_CLIENT_CONF_PATH` environment variable for minicluster * Update SaslServerConnectionParams to remove unused overloaded constructors and setters (allowing secretManager field to be final); this wasn't strictly necessary in this patch, but helped simplify things to make it easier to trace the uses of the ClientConfConverter methods to see how they were still needed -- 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]
