Github user jasobrown commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/184#discussion_r163421217
--- 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 --
ughh, so I was completely wrong wrt the RPC_ADDRESS/NATIVE_ADDRESS thing. i
misread this line:
```java
appStates.put(ApplicationState.RPC_ADDRESS,
valueFactory.rpcaddress(FBUtilities.getBroadcastRpcAddress()));
```
as this:
```java
appStates.put(ApplicationState.RPC_ADDRESS,
valueFactory.rpcaddress(FBUtilities.getBroadcastAddress()));
```
(`getBroadcastRpcAddress` vs `getBroadcastAddress`) when we do the
assignment in StorageService.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
