Github user ifesdjeen commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/224#discussion_r197128495
--- Diff:
src/java/org/apache/cassandra/locator/AbstractReplicationStrategy.java ---
@@ -329,6 +335,10 @@ public static void validateReplicationStrategy(String
keyspaceName,
AbstractReplicationStrategy strategy =
createInternal(keyspaceName, strategyClass, tokenMetadata, snitch,
strategyOptions);
strategy.validateExpectedOptions();
strategy.validateOptions();
+ if (strategy.getReplicationFactor().trans > 0 &&
!DatabaseDescriptor.isTransientReplicationEnabled())
--- End diff --
Check for version. Previously we had allNodesAtLeast22/30 for some internal
handling, so we might have to implement similar mechanism for that.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
