kevinrr888 commented on code in PR #5678:
URL: https://github.com/apache/accumulo/pull/5678#discussion_r2175482702
##########
core/src/main/java/org/apache/accumulo/core/data/TabletId.java:
##########
@@ -26,6 +28,62 @@
* @since 1.7.0
*/
public interface TabletId extends Comparable<TabletId> {
+
+ /**
+ * Return a TabletId object for the provided TableId in range of
(prevEndRow, endRow]. If the
+ * parameters endRow and/or prevEndRow are null, they represent a tablet
over the range of -inf
+ * and/or inf respectively.
Review Comment:
```suggestion
* parameters prevEndRow and/or endRow are null, they represent a tablet
over the range of -inf
* and/or inf respectively.
```
These should be flipped to align with the range mentioned here
##########
core/src/main/java/org/apache/accumulo/core/data/TabletId.java:
##########
@@ -26,6 +28,62 @@
* @since 1.7.0
*/
public interface TabletId extends Comparable<TabletId> {
+
+ /**
+ * Return a TabletId object for the provided TableId in range of
(prevEndRow, endRow]. If the
+ * parameters endRow and/or prevEndRow are null, they represent a tablet
over the range of -inf
+ * and/or inf respectively.
+ *
+ * @param tableId the ID for a table
+ * @param endRow the last row in this tablet, or null if this is the last
tablet in this table
+ * @param prevEndRow he last row in the immediately preceding tablet for the
table, or null if
Review Comment:
```suggestion
* @param prevEndRow the last row in the immediately preceding tablet for
the table, or null if
```
--
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]
