mike-tr-adamson commented on code in PR #2673:
URL: https://github.com/apache/cassandra/pull/2673#discussion_r1371668822
##########
src/java/org/apache/cassandra/index/sai/disk/PrimaryKeyMap.java:
##########
@@ -70,6 +71,22 @@ default void close()
*/
long rowIdFromPrimaryKey(PrimaryKey key);
+ /**
+ * Returns the first row ID for a given token or a negative value if not
found
+ *
+ * @param token the {@link Token} to lookup
+ * @return the first row ID associated with the {@link Token} or a
negative value
+ */
+ long ceiling(Token token);
+
+ /**
+ * Returns the last row ID for a given partition range or the previous row
ID if there was no match for the partition range
Review Comment:
I have changed the javadoc for these as follows:
- ceiling - Returns the first row ID of the nearest {@link Token} greater
than or equal to the given {@link Token}, or a negative value if not found
- floor - Returns the last row ID of the nearest {@link Token} less than or
equal to the given {@link Token}, or a negative value if the {@link Token} is
at its minimum value
I believe this is the closest to reality.
--
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]
