benibus commented on code in PR #35798:
URL: https://github.com/apache/arrow/pull/35798#discussion_r1228415712


##########
cpp/src/arrow/dataset/file_parquet_test.cc:
##########
@@ -694,5 +694,55 @@ TEST(TestParquetStatistics, NullMax) {
   EXPECT_EQ(stat_expression->ToString(), "(x >= 1)");
 }
 
+// Tests round-trip projection with nested/indexed FieldRefs
+// https://github.com/apache/arrow/issues/35579
+TEST(TestRoundTrip, ProjectedFieldRefs) {

Review Comment:
   Yeah, I only did it this way to more or less match the example in the 
original issue. Using the existing fixtures would certainly be more convenient. 
AFAIK the only difference is that it would test the intermediate batch rather 
than doing the full round trip on the projection - but that's consistent with 
the other tests.



-- 
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]

Reply via email to