[GitHub] [arrow] jorisvandenbossche commented on pull request #34137: GH-34136: [C++] Add a concept of ordering to ExecPlan

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

jorisvandenbossche commented on PR #34137:
URL: https://github.com/apache/arrow/pull/34137#issuecomment-1439130374

   > One more question: assume you would use something like `DeclarationToTable`, does that automatically use the ordering / batch indices if there is one, or do you still need to indicate this manually you want to use it? (like the generic `SinkNodeOptions` has a `sequence_delivery` parameter with a default of false, but I don't see that exposed through the DeclarationTo.. versions)
   
   Ah, I see there is a new variant of `DeclarationToTable` that uses QueryOptions as parameter, and that options struct has a `sequence_output` keyword. So I assume that answers my question.
   
   Follow-up question on this: the default for `sequence_output` is false, so does that mean that even if you have a query plan with an order_by node at the end before consuming as eg a table, you won't get the data as ordered by default, because for the ordering to be respected, you need to explicitly enable it with `sequence_output`?
   
   


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