ctubbsii commented on a change in pull request #1379: Improved metadata table 
consitency checking and testing
URL: https://github.com/apache/accumulo/pull/1379#discussion_r329727021
 
 

 ##########
 File path: 
server/base/src/main/java/org/apache/accumulo/server/util/TabletIterator.java
 ##########
 @@ -101,6 +102,20 @@ public boolean hasNext() {
 
       currentTabletKeys = scanToPrevEndRow();
       if (currentTabletKeys.size() == 0) {
+        if (lastTablet != null) {
 
 Review comment:
   A brief human-friendly comment might be helpful to explain what this block 
is trying to check for:
   
   ```suggestion
           // Always expect the default tablet to exist for a table. The 
following checks for the case when
           // the default tablet was not seen when it should have been seen.
           if (lastTablet != null) {
   ```
   
   Comment copied from #1266 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to