comphead commented on code in PR #8620:
URL: https://github.com/apache/arrow-datafusion/pull/8620#discussion_r1435665526
##########
.github/workflows/rust.yml:
##########
@@ -99,6 +99,11 @@ jobs:
rust-version: stable
- name: Run tests (excluding doctests)
run: cargo test --lib --tests --bins --features avro,json,backtrace
+ env:
Review Comment:
Adding more stack as q54 keeps failing.
```
thread 'tpcds_physical_q54' has overflowed its stack
```
This PR shouldn't impact stack as the introduced `SchemaError` second param
is boxed and thus resides in heap, in stack there is only a pointer. However it
is what is is, I'm happy to get the opinion from other contributors on this
--
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]
