elek commented on a change in pull request #536: HDDS-2988. Use
getPropertiesByPrefix instead of regex in matching ratis client and server
properties.
URL: https://github.com/apache/hadoop-ozone/pull/536#discussion_r377102805
##########
File path:
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/ratis/RatisHelper.java
##########
@@ -54,29 +57,34 @@
import org.apache.ratis.retry.RetryPolicy;
import org.apache.ratis.rpc.RpcType;
import org.apache.ratis.rpc.SupportedRpcType;
+import org.apache.ratis.server.RaftServerConfigKeys;
import org.apache.ratis.thirdparty.com.google.protobuf.ByteString;
import org.apache.ratis.util.TimeDuration;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import static
org.apache.hadoop.ozone.conf.DatanodeRatisServerConfig.DATANODE_RATIS_SERVER_CONFIG_PREFIX;
-
/**
* Ratis helper methods.
*/
public interface RatisHelper {
Logger LOG = LoggerFactory.getLogger(RatisHelper.class);
- // Ratis Client and Grpc header regex filters.
- String RATIS_CLIENT_HEADER_REGEX = "raft\\.client\\.([a-z\\.]+)";
- String RATIS_GRPC_CLIENT_HEADER_REGEX = "raft\\.grpc\\.(?!server|tls)" +
- "([a-z\\.]+)";
+ // Prefix for Ratis Server GRPC and Ratis client conf.
+ String HDDS_DATANODE_RATIS_PREFIX_KEY = "datanode.ratis.";
Review comment:
Nit: `HDDS_DATANODE_RATIS_PREFIX_KEY` can be reused.
```
String HDDS_DATANODE_RATIS_PREFIX_KEY = "datanode.ratis.";
String HDDS_DATANODE_RATIS_SERVER_PREFIX_KEY =
HDDS_DATANODE_RATIS_PREFIX_KEY + ".server";
String HDDS_DATANODE_RATIS_CLIENT_PREFIX_KEY =
HDDS_DATANODE_RATIS_PREFIX_KEY + ".client";
String HDDS_DATANODE_RATIS_GRPC_PREFIX_KEY =
HDDS_DATANODE_RATIS_PREFIX_KEY + ".grpc";
```
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org
