You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2021/11/08 19:24:14 UTC

[GitHub] [hudi] nsivabalan commented on a change in pull request #3947: [HUDI-2712] Fixing list based rollback of partially failed commit which has new partitions

nsivabalan commented on a change in pull request #3947:
URL: https://github.com/apache/hudi/pull/3947#discussion_r745026263



##########
File path: hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/functional/TestHoodieBackedMetadata.java
##########
@@ -276,9 +277,8 @@ public void testInsertUpsertCluster(HoodieTableType tableType) throws Exception
   /**
    * Test rollback of various table operations sync to Metadata Table correctly.
    */
-  //@ParameterizedTest
-  //@EnumSource(HoodieTableType.class)
-  @Disabled
+  @ParameterizedTest
+  @EnumSource(HoodieTableType.class)

Review comment:
       this test along with testErrorCases was flaky in CI and to unblock CI disabled it sometime back. now with this fix, should not be flaky anymore. 




-- 
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: commits-unsubscribe@hudi.apache.org

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