yashmayya commented on code in PR #15703:
URL: https://github.com/apache/pinot/pull/15703#discussion_r2072833382


##########
pinot-common/src/main/java/org/apache/pinot/common/cursors/AbstractResponseStore.java:
##########
@@ -185,7 +185,7 @@ public CursorResponse handleCursorRequest(String requestId, 
int offset, int numR
     int totalTableRows = response.getNumRowsResultSet();
 
     if (totalTableRows == 0 && offset == 0) {
-      // If sum records is 0, then result set is empty.
+      // TODO: Set a result table with schema and no rows

Review Comment:
   This is in order to match the cursor response behavior with the regular 
query response behavior after this PR IIUC?



-- 
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: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to