dcapwell commented on code in PR #4202: URL: https://github.com/apache/cassandra/pull/4202#discussion_r2155065069
########## src/java/org/apache/cassandra/utils/ImmutableUniqueList.java: ########## @@ -107,6 +119,23 @@ public int size() return values.length; } + @Override + public boolean equals(Object o) Review Comment: AST model depends on checking existence with a set, so support defining equals with set based just on the `containsAll` property rather than the order property. -- 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: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org