narendly commented on a change in pull request #1178:
URL: https://github.com/apache/helix/pull/1178#discussion_r470285822
##########
File path:
zookeeper-api/src/main/java/org/apache/helix/zookeeper/api/client/RealmAwareZkClient.java
##########
@@ -486,6 +500,13 @@ private void validate() {
throw new IllegalArgumentException(
"RealmAwareZkConnectionConfig.Builder: ZK sharding key must be
set on single-realm mode!");
}
+ if ((_routingDataSourceEndpoint == null && _routingDataSourceType !=
null) || (
+ _routingDataSourceEndpoint != null && _routingDataSourceType ==
null)) {
+ // For routing data source type and endpoint, if one is set and not
the other, it is invalid
Review comment:
Then they shouldn't be using `RealmAwareZkClient` in that case since
they aren't using multiple zookeepers.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
