bereng commented on code in PR #2125:
URL: https://github.com/apache/cassandra/pull/2125#discussion_r1106692062
##########
test/distributed/org/apache/cassandra/distributed/upgrade/MixedModeColumnMaskingTest.java:
##########
@@ -104,10 +111,21 @@ private static void assertFails(ICoordinator coordinator,
String schemaQuery, St
.hasMessageContaining(expectedMessage);
}
- private void assertColumnValue(UpgradeableCluster cluster, String table,
String column, Object value)
+ private static void assertColumnIsMasked(UpgradeableCluster cluster,
String table, String column)
Review Comment:
I would do it if there was an easy way to target the SELECT to a specific
node, but missing that it's not worth it imo
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
