nealrichardson commented on issue #34437:
URL: https://github.com/apache/arrow/issues/34437#issuecomment-1479559419
I tried just removing the validation in the fetch node like this:
```
--- a/cpp/src/arrow/compute/exec/fetch_node.cc
+++ b/cpp/src/arrow/compute/exec/fetch_node.cc
@@ -126,12 +126,6 @@ class FetchNode : public ExecNode, public TracedNode,
util::SequencingQueue::Pro
Status Validate() const override {
ARROW_RETURN_NOT_OK(ExecNode::Validate());
- if (inputs_[0]->ordering().is_unordered()) {
- return Status::Invalid(
- "Fetch node's input has no meaningful ordering and so
limit/offset will be "
- "non-deterministic. Please establish order in some way (e.g. by
inserting an "
- "order_by node)");
- }
return Status::OK();
}
```
and that caused the tests to hang.
--
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]
