You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/24 10:28:03 UTC

[GitHub] [arrow-datafusion] alamb commented on pull request #2521: Support for OFFSET in LogicalPlan

alamb commented on PR #2521:
URL: https://github.com/apache/arrow-datafusion/pull/2521#issuecomment-1135736282

   Hi @ming535  -- it depends on how the operators work
   
   Of limit is the input to offset, I think you can get the right answers. 
   
   For example, with `limit 10 offset 5` the idea would be that the `LIMIT`  produces 15 rows, and then the offset skips the first 5, resulting in 10 in total.
   
   I realize this is not what the planner does now (it doesn't adjust the limit to 15, it uses 10). However, the plan also doesn't run as the physical plan for offset isn't completed. I will file a ticket for that


-- 
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: github-unsubscribe@arrow.apache.org

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