ming535 commented on issue #2624: URL: https://github.com/apache/arrow-datafusion/issues/2624#issuecomment-1138768411
How about add another LogicalPlan `LimitWithOffset`. So that the three LogicalPlan used in different cases: 1. `Limit`: without offset 2. `Offset`: without limit 3. `LimitWithOffset`: offset + limit Then in the `limit_push_down`, we need to handle two cases: `Limit` and `LimitWithOffset`. For `Limit`, the push down logic is roughly the same as before; for `LimitWithOffset`, we adjust the limit pushed down. -- 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]
