ajreid21 commented on code in PR #14565:
URL: https://github.com/apache/iceberg/pull/14565#discussion_r2517058536


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -4402,6 +4402,11 @@ components:
           description:
             Expression used to filter the table data
           $ref: '#/components/schemas/Expression'
+        min-rows-requested:
+          description:
+            The minimum number of rows requested for the scan

Review Comment:
   Yes, this is intended as a "hint" or "indicator" from the client to help the 
server not have to return more than is necessary. It is not required for the 
server to return that many results (as the result of the scan may not have that 
many rows, that's why it's named "min-rows-requested" and not 
"min-rows-required"). Also, it's not a max limit either and the server can 
return more than the requested number.
   
   I'm open to different name and better description for this



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to