Apache9 commented on code in PR #6900:
URL: https://github.com/apache/hbase/pull/6900#discussion_r2053692283


##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java:
##########
@@ -832,7 +845,7 @@ private void updateMetricsStore(boolean memstoreRead) {
    * @return null is the top cell doesn't change. Otherwise, the NextState to 
return
    */
   private NextState needToReturn(List<? super ExtendedCell> outResult) {
-    if (!outResult.isEmpty() && topChanged) {
+    if (topChanged) {

Review Comment:
   What if there are no data for row1 in cf2? What is the current way to 
prevert loading row2?
   
   I mean we have delete row1 in cf1, and row2 in cf1 and cf2, how can we make 
sure that we do not return row2 when calling cf2.next at the first time?



-- 
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]

Reply via email to