You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "hudi-bot (via GitHub)" <gi...@apache.org> on 2025/11/30 13:44:58 UTC

[I] Re-enable TestClusteringUtils#testClusteringPlanInflight [hudi]

hudi-bot opened a new issue, #17333:
URL: https://github.com/apache/hudi/issues/17333

   ## JIRA info
   
   - Link: https://issues.apache.org/jira/browse/HUDI-8686
   - Type: Sub-task
   - Parent: https://issues.apache.org/jira/browse/HUDI-8875
   - Fix version(s):
     - 1.1.0


-- 
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.apache.org

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


Re: [I] Re-enable TestClusteringUtils#testClusteringPlanInflight [hudi]

Posted by "voonhous (via GitHub)" <gi...@apache.org>.
voonhous closed issue #17333: Re-enable TestClusteringUtils#testClusteringPlanInflight
URL: https://github.com/apache/hudi/issues/17333


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


Re: [I] Re-enable TestClusteringUtils#testClusteringPlanInflight [hudi]

Posted by "rangareddy (via GitHub)" <gi...@apache.org>.
rangareddy commented on issue #17333:
URL: https://github.com/apache/hudi/issues/17333#issuecomment-5351233261

   This issue was reviewed as part of the JIRA-migrated backlog triage (HUDI-8686).
   
   **Findings: still disabled, but the annotation's own stated blocker is already resolved.**
   
   `hudi-hadoop-common/src/test/java/org/apache/hudi/common/util/TestClusteringUtils.java:176`:
   
   ```java
   // replacecommit.inflight doesn't have clustering plan.
   // Verify that getClusteringPlan fetches content from corresponding requested file.
   @Disabled("Will fail due to avro issue AVRO-3789. This is fixed in avro 1.11.3")
   @Test
   public void testClusteringPlanInflight() throws Exception {
   ```
   
   The annotation says the fix is in avro 1.11.3. The root `pom.xml` now pins `<avro.version>1.11.4</avro.version>`, which is past that. Unlike the other re-enable tickets in this batch, `@Test` is present here, so removing the single `@Disabled` line should be enough.
   
   This looks like a one-line change plus a CI run. Keeping it open only until someone confirms it passes.
   


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