Github user aweisberg commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/184#discussion_r163304964
--- Diff: src/java/org/apache/cassandra/gms/ApplicationState.java ---
@@ -19,24 +19,25 @@
public enum ApplicationState
{
- STATUS,
+ @Deprecated STATUS, //Deprecated and unsued in 4.0, stop publishing in
5.0, reclaim in 6.0
LOAD,
SCHEMA,
DC,
RACK,
RELEASE_VERSION,
REMOVAL_COORDINATOR,
- INTERNAL_IP,
- RPC_ADDRESS,
+ @Deprecated INTERNAL_IP, //Deprecated and unused in 4.0, stop
publishing in 5.0, reclaim in 6.0
+ @Deprecated RPC_ADDRESS, // ^ Same
X_11_PADDING, // padding specifically for 1.1
SEVERITY,
NET_VERSION,
HOST_ID,
TOKENS,
RPC_READY,
// pad to allow adding new states to existing cluster
- X1,
- X2,
+ INTERNAL_ADDRESS_AND_PORT, //Replacement for INTERNAL_IP with up to
two ports
+ NATIVE_ADDRESS_AND_PORT, //Replacement for RPC_ADDRESS
--- End diff --
native address is what it is now because thrift RPC is gone. Now what I
remember is I did this because I thought we stopped using rpc_address in the
yaml and renamed it to native_address. And now I am very confused because that
doesn't seem to be the case. I was trying to consistently use native_address
instead of rpc_address now that thrift is gone.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
