yihua commented on code in PR #13515:
URL: https://github.com/apache/hudi/pull/13515#discussion_r2363513198


##########
hudi-hadoop-common/src/test/java/org/apache/hudi/common/table/log/block/TestHoodieDeleteBlock.java:
##########
@@ -54,6 +59,20 @@ public class TestHoodieDeleteBlock {
 
   private static Random random = new Random();
 
+  @Test
+  void validateHoodieDeleteRecordListFieldsAndOrdinals() {
+    // HoodieDeleteBlock uses IndexedRecord instead of HoodieDeleteRecordList 
as the output of
+    // reading the delete record list, due to class loading issue on the 
executor side on Spark

Review Comment:
   Spark 3.4.x also supports Java 17 and the SPARK-51537 is not cherrypicked to 
Spark 3.4 branch so there is still problem with the latest Spark 3.4.  If we 
want to give the guideline that for Hudi on Java 17 the user should only use 
latest Spark 3.5, not Spark 3.4, then we don't need this fix and we'll update 
the documentation on such guideline.



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