Yukei7 commented on code in PR #3754:
URL: https://github.com/apache/cassandra/pull/3754#discussion_r1945742217
##########
src/java/org/apache/cassandra/db/lifecycle/View.java:
##########
@@ -353,4 +361,19 @@ public boolean apply(T t)
}
};
}
+
+ private static int getSSTablesHash(Iterable<SSTableReader> readers) {
+ int hashSum = 0;
Review Comment:
updated this method a bit - instead of calculating the hash, it's probably
better to find a 1-1 mapping from `toAdd` and `toRemove`. If their range is the
same and such 1-1 mapping exists, we can simply replaced those.
let me know if this works or you have better idea here
--
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]
