Skip to content

Conversation

@NGA-TRAN
Copy link
Collaborator

@NGA-TRAN NGA-TRAN commented Jul 1, 2025

This PR adds a reproducer for TPC-H Q16, as reported in issue #33.
Note: It also triggers the Q15 bug, which is addressed (but not yet merged) in PR #32. However, this PR focuses exclusively on Q16.

Summary of the pro:

cargo test --package distributed-datafusion --lib -- tpch_tests::tpch_serialization_tests::test_all_tpch_queries_serialization --exact --show-output 

🔍 Testing query: q16
✅ Serialization successful for q16
❌ Deserialization failed for q16: Internal error: The data type inlist should be same, the value type is Utf8View, one of list expr type is Int32.
This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker
❌ q16 failed: Internal error: The data type inlist should be same, the value type is Utf8View, one of list expr type is Int32.
This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker

@NGA-TRAN NGA-TRAN marked this pull request as draft July 1, 2025 21:20
@NGA-TRAN
Copy link
Collaborator Author

Close this PR without merging because we already reproduced it upstream apache/datafusion#16662 and created a upstream bug ticket apache/datafusion#16665

I also added more thorough tests upstream apache/datafusion#16742.

All the fixed will be done upstream

@NGA-TRAN NGA-TRAN closed this Jul 10, 2025
@gabotechs gabotechs deleted the fix/q16-in-expression-type-coercion branch August 4, 2025 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants