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/08/26 16:23:47 UTC
[GitHub] [arrow-datafusion] DaltonModlin opened a new issue, #3273: View doesn't populate session context table_names
DaltonModlin opened a new issue, #3273:
URL: https://github.com/apache/arrow-datafusion/issues/3273
**Describe the bug**
Currently, running Q15 fails when attempting to build the second (SELECT) portion of the overall query.
The error is similar to the following:
```
called `Result::unwrap()` on an `Err` value: Plan("'datafusion.public.revenue0' not found")
thread 'tests::run_q15' panicked at 'called `Result::unwrap()` on an `Err` value: Plan("'datafusion.public.revenue0' not found")', /home/dalto/projects/arrow-datafusion/datafusion/sql/src/planner.rs:161:31
```
**To Reproduce**
Run test run_q15 in `benchmarks/src/bin/tpch.rs`
**Expected behavior**
I expect the test to fail due to either #3266 or #3267
**Additional context**
The table names just before failure are:
```
schema table names: [
"customer",
"supplier",
"region",
"part",
"orders",
"nation",
"lineitem",
"partsupp",
]
```
I'm unsure of where view's populate this table, if anyone else has knowledge here that would be great.
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] DaltonModlin commented on issue #3273: View doesn't populate session context table_names
Posted by GitBox <gi...@apache.org>.
DaltonModlin commented on issue #3273:
URL: https://github.com/apache/arrow-datafusion/issues/3273#issuecomment-1228710345
That's exactly what I was thinking looking at it last night, and I was just writing a test to confirm by splitting the query. @kmitchener good to know I'm moving in the right direction. Assuming I get a different error by splitting the query, I'll try and rewrite the testing processing in tpch.rs and see if it fixes this problem.
--
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
[GitHub] [arrow-datafusion] avantgardnerio commented on issue #3273: View doesn't populate session context table_names
Posted by GitBox <gi...@apache.org>.
avantgardnerio commented on issue #3273:
URL: https://github.com/apache/arrow-datafusion/issues/3273#issuecomment-1228925425
> Probably run_query() should be modified to execute the statements in turn, rather than attempting to plan all of them before beginning execution.
This :point_up: . AFAIK, the TPC-H queries are what they are, and we have to run them as such. This one is ensuring we support multiple queries in one command, so to be TPC-H compliant I think it has to "just work".
--
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
[GitHub] [arrow-datafusion] kmitchener commented on issue #3273: View doesn't populate session context table_names
Posted by GitBox <gi...@apache.org>.
kmitchener commented on issue #3273:
URL: https://github.com/apache/arrow-datafusion/issues/3273#issuecomment-1228705128
This is happening because of the way tpch is running the test queries. q15 is made up of 3 SQL statements. run_query() attempts to create a plan for each of them before executing them. The second statement in q15 references the revenue0 view created in the first statement (which isn't created yet), so the planning of the second statement fails.
Probably run_query() should be modified to execute the statements in turn, rather than attempting to plan all of them before beginning execution.
--
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
[GitHub] [arrow-datafusion] andygrove closed issue #3273: Execute LogicalPlans after building for TPCH Benchmarks
Posted by GitBox <gi...@apache.org>.
andygrove closed issue #3273: Execute LogicalPlans after building for TPCH Benchmarks
URL: https://github.com/apache/arrow-datafusion/issues/3273
--
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
[GitHub] [arrow-datafusion] avantgardnerio commented on issue #3273: View doesn't populate session context table_names
Posted by GitBox <gi...@apache.org>.
avantgardnerio commented on issue #3273:
URL: https://github.com/apache/arrow-datafusion/issues/3273#issuecomment-1228694864
@kmitchener @andygrove @alamb
--
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