virajjasani commented on PR #2083: URL: https://github.com/apache/phoenix/pull/2083#issuecomment-2683896585
@stoty this change is just restoring the version of what old client used to return. This is for backward compatibility purpose. Imagine, a downstream application using dual client connection using both 5.1 and 5.2 clients and expecting CQSI config to return same value for `hbase.zookeeper.quorum` depending on the primary/secondary cluster pairs defined somewhere. Custom property is fine but by not doing this, not only we break compatibility, we return incorrect value of zk quorum config because unless overriden by ZkConnectionInfo, the value will be derived from cached config in ConnectionInfo. -- 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]
