openinx commented on a change in pull request #3250:
URL: https://github.com/apache/iceberg/pull/3250#discussion_r725757723
##########
File path: data/src/test/java/org/apache/iceberg/io/TestFileWriterFactory.java
##########
@@ -131,8 +133,6 @@ public void testDataWriter() throws IOException {
@Test
public void testEqualityDeleteWriter() throws IOException {
- Assume.assumeFalse("ORC delete files are not supported", fileFormat ==
FileFormat.ORC);
-
Review comment:
Thanks for the work, Making all the tests works fine without the ORC
exclusion does definitely proof the ORC write path works as expected !
--
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]
