ajantha-bhat commented on code in PR #8098:
URL: https://github.com/apache/iceberg/pull/8098#discussion_r1379600138


##########
parquet/src/test/java/org/apache/iceberg/avro/TestParquetReadProjection.java:
##########
@@ -45,4 +45,20 @@ protected GenericData.Record writeAndRead(
 
     return Iterables.getOnlyElement(records);
   }
+
+  // Empty struct read is not supported for Parquet
+  @Override
+  public void testEmptyStructProjection() throws Exception {}

Review Comment:
   These tests was added for Avro. When empty struct was supported. But it is 
not yet supported for parquet. Since both avro and parquet now use the common 
class. Ignoring the tests for parquet. 



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to