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


##########
hudi-spark-datasource/hudi-spark/src/test/java/org/apache/hudi/functional/TestGlobalIndexEnableUpdatePartitions.java:
##########
@@ -215,17 +232,17 @@ public void 
testRollbacksWithPartitionUpdate(HoodieTableType tableType, IndexTyp
       String commitTimeAtEpoch15 = TimelineUtils.generateInstantTime(false, 
timeGenerator);
       List<HoodieRecord> updatesAtEpoch15 = getUpdates(updatesAtEpoch5, p3, 
15, payloadClass);
       WriteClientTestUtils.startCommitWithTime(client, commitTimeAtEpoch15);
-      assertNoWriteErrors(client.upsert(jsc().parallelize(updatesAtEpoch15, 
2), commitTimeAtEpoch15).collect());
-      // for the same bug pointed out earlier, (ignoring rollbacks while 
determining last instant while reading log records), this tests the 
HoodieFileGroupReader.
+      client.commit(commitTimeAtEpoch15, 
client.upsert(jsc().parallelize(updatesAtEpoch15, 2), commitTimeAtEpoch15));
+      // for the same bug pointed out earlier, (ignoring rollbacks while 
determining last instant while reading log records), this tests the 
HoodieMergedReadHandle.

Review Comment:
   i.e., `HoodieMergedReadHandle` no longer exists



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