Re: [PR] Limit pages size to a configurable limit (druid)

["marzi312 (via GitHub)" <gi...@apache.org>]

marzi312 commented on PR #14994:
URL: https://github.com/apache/druid/pull/14994#issuecomment-1959955932

   hey @cryptoe 
   I've tested this feature with a curl command, POST to ``/v2/sql/statements`` but it completely ignores `rowsPerPage ` parameter. 
   Example: 
   `curl -XPOST -H "Content-Type: application/json" -d '{"context": {"includeSegmentSource":"REALTIME","selectDestination":"DURABLESTORAGE", "executionMode":"ASYNC", "durableShuffleStorage": "true", "rowsPerPage":"1000"}, "query":"SELECT * FROM my_table limit 200000"}' https://<url>/druid/v2/sql/statements
   `
   
   `rowsPerPage` default value is 100000 but it seems it even ignores the default value for me. 
    I tested this with Druid V29 
    
    and here's the results from get query status endpoint
   `...."pages":[{"id":0,"numRows":200000,"sizeInBytes":123026540}]}`
   


-- 
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-unsubscribe@druid.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org