adityaanikam commented on code in PR #19653:
URL: https://github.com/apache/hudi/pull/19653#discussion_r3803682280


##########
hudi-flink-datasource/hudi-flink/src/main/java/org/apache/hudi/util/ClientIds.java:
##########
@@ -219,6 +219,14 @@ private static String getClientId(Path path) {
     return splits.length > 1 ? splits[1] : INIT_CLIENT_ID;
   }
 
+  /**
+   * Returns a sort key so heartbeat files order numerically instead of 
lexicographically.
+   * The base heartbeat file (empty client id) always sorts first.
+   */
+  private static int getClientIdSortKey(String clientId) {

Review Comment:
   The failure is unrelated to this change the single test failure is Test Call 
sync_validate record count modes fail fast on an unreachable HiveServer2 in the 
Spark Hive sync procedure suite, a network dependent test. Azure's own log even 
marks it as flaky and excludes it from the pass percentage. Triggering a rerun.
   
   @hudi-bot run azure



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