qqu0127 commented on code in PR #2291:
URL: https://github.com/apache/helix/pull/2291#discussion_r1033689120
##########
meta-client/src/main/java/org/apache/helix/metaclient/factories/MetaClientConfig.java:
##########
@@ -56,56 +69,81 @@ public StoreType getStoreType() {
// private RetryProtocol _retryProtocol;
- private MetaClientConfig(String connectionAddress, long connectionTimeout,
boolean enableAuth,
- StoreType storeType) {
+ protected MetaClientConfig(String connectionAddress, long
connectionInitTimeout,
+ long sessionTimeout, boolean enableAuth, StoreType storeType) {
_connectionAddress = connectionAddress;
- _connectionTimeout = connectionTimeout;
+ _connectionInitTimeout = connectionInitTimeout;
+ _sessionTimeout = sessionTimeout;
_enableAuth = enableAuth;
_storeType = storeType;
}
- public static class Builder {
- private String _connectionAddress;
-
- private long _connectionTimeout;
- private boolean _enableAuth;
- //private RetryProtocol _retryProtocol;
- private StoreType _storeType;
+ public static class MetaClientConfigBuilder<B extends
MetaClientConfigBuilder<B>> {
Review Comment:
Thanks, that makes sense. This seems to be a very good use of generics.
Can we make this class standalone? It's a `public static` anyway, that way
might be more clear.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
