viirya opened a new issue, #1113:
URL: https://github.com/apache/datafusion-comet/issues/1113
### Describe the bug
Currently we get the partition index from TaskContext in CometExecIterator.
It is actually incorrect. For example, in a query like
```
CartesianProductExec (25 partitions)
- CometProject
- CometScan (5 partitions)
- CometProject
- CometScan (5 partitions)
```
There are 25 partitions in the task for the zipped partitions from both
CometScan's 5 partitions. So the partition indexes are 0~24. Each CometProject
+ CometScan has represented by a CometExecIterator. When executing the native
plan, these partition indexes are used. However, it only has 5 partitions.
Currently this doesn't exposed as test failure because the partition index
is not actually used.
But in the feature branch `comet-parquet-exec`, the native ParquetExec uses
given partition index to access partitioned file in file group. The incorrect
partition index causes errors like out of index.
### Steps to reproduce
_No response_
### Expected behavior
_No response_
### Additional context
_No response_
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
