Zouxxyy commented on PR #8955:
URL: https://github.com/apache/hudi/pull/8955#issuecomment-1624527608

   > Looks like only the order of column is wrong, but could you help me 
understand if this is a valid failure or we should fix the test?
   
   Are you testing java17? https://github.com/apache/hudi/pull/9136/files#top
   It seems that the order of items in set in java17 has changed, we can change 
the test case like this, if we need to support java17
   
   from
   
   ```java
       assertEquals("Field fake_field not found in log schema. Query cannot 
proceed! Derived Schema Fields: "
               + "[non_pii_col, _hoodie_commit_time, _row_key, 
_hoodie_partition_path, _hoodie_record_key, pii_col,"
               + " _hoodie_commit_seqno, _hoodie_file_name, timestamp]",
           assertThrows(HoodieException.class, () ->
               HoodieAvroUtils.generateProjectionSchema(originalSchema, 
Arrays.asList("_row_key", "timestamp", "fake_field"))).getMessage());
   ```
   to
   
   ```java
        assertTrue(assertThrows(HoodieException. class, () ->
            HoodieAvroUtils.generateProjectionSchema(originalSchema, 
Arrays.asList("_row_key", "timestamp", "fake_field")))
            .getMessage().contains("Field fake_field not found in log schema. 
Query cannot proceed!"));
   ```


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