michaelsembwever commented on PR #2336: URL: https://github.com/apache/cassandra/pull/2336#issuecomment-1549752738
can you please start with this commit: https://github.com/apache/cassandra/compare/trunk...thelastpickle:cassandra:mck/18506/trunk this fixes the `.contains(..)` bug, and should make things a lot clearer to understand, without having to touch method signatures (which touches all the other classes). starting off like this will help the review process progress (we can re-evaluate the need to improve method names later on in the review process…) -- 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]
