keith-turner commented on code in PR #5833:
URL: https://github.com/apache/accumulo/pull/5833#discussion_r2307800408
##########
core/src/main/java/org/apache/accumulo/core/file/rfile/RFile.java:
##########
@@ -1662,23 +1625,8 @@ public Value getTopValue() {
}
@Override
- public Text getFirstRow() throws IOException {
Review Comment:
This is where one of the two bugs was. This method and getLastRow could
compute a last row that was before the start row when the fence did not overlap
the first and last row in the file. Now the new code should return an empty
FileRange for this case.
--
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: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
