gabotechs commented on code in PR #16662:
URL: https://github.com/apache/datafusion/pull/16662#discussion_r2181971781
##########
datafusion/proto/tests/cases/roundtrip_physical_plan.rs:
##########
@@ -1736,3 +1737,57 @@ async fn roundtrip_physical_plan_node() {
let _ = plan.execute(0, ctx.task_ctx()).unwrap();
}
+
+#[tokio::test]
+async fn test_tpch_part_in_list_query_with_real_parquet_data() -> Result<()> {
+ // Test the specific query: SELECT p_size FROM part WHERE p_size IN (14,
6, 5, 31)
+ // This uses REAL TPC-H parquet data to reproduce the serialization bug
with actual data sources
+ //
+ // NOTE: This test requires TPC-H data to reproduce the serialization bug.
+ // Using alltypes_plain.parquet does NOT reproduce the issue, suggesting
the bug
+ // is specific to certain characteristics of TPC-H parquet files or their
schema.
+
+ // Check if TPC-H data is available
+ if !Path::new("/tmp/tpch_s1/part.parquet").exists() {
Review Comment:
Does the CI dump tpch data in `/tmp`? if it does not do it this test might
never be enforced in any pipeline. I see there's a `testdata` folder near these
tests, maybe we can dump a truncated version of the parquet file there?
--
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]
