FMX commented on code in PR #2003:
URL:
https://github.com/apache/incubator-celeborn/pull/2003#discussion_r1365440208
##########
common/src/main/java/org/apache/celeborn/common/protocol/RpcNameConstants.java:
##########
@@ -19,18 +19,19 @@
public class RpcNameConstants {
// For Master
- public static String MASTER_SYS = "MasterSys";
-
+ public static String MASTER_SYS = "Master";
// Master Endpoint Name
public static String MASTER_EP = "MasterEndpoint";
// For Worker
- public static String WORKER_SYS = "WorkerSys";
Review Comment:
What will happen if the server is using this PR and the client is old? Will
this name change introduce a compatible issue?
--
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]
