jpisaac commented on code in PR #2138:
URL: https://github.com/apache/phoenix/pull/2138#discussion_r2087321697
##########
phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java:
##########
@@ -478,31 +490,65 @@ public ConnectionQueryServicesImpl(QueryServices
services, ConnectionInfo connec
// Without making a copy of the configuration we cons up, we lose some
of our properties
// on the server side during testing.
- this.config =
HBaseFactoryProvider.getConfigurationFactory().getConfiguration(config);
+ Configuration finalConfig = HBaseFactoryProvider
Review Comment:
Can we add some comments to the effect, that this allows the application
overridden ConfigurationFactory to inject/modify configs
--
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]
