[HUDI-4951] Fix incorrect use of Long.getLong() (#6828)

shenshengli · web-flow · commit bfb4946f3997 · 2022-09-29T08:17:15.000-07:00
diff --git a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/BaseHoodieWriteClient.java b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/BaseHoodieWriteClient.java
@@ -1694,7 +1694,7 @@ private void commitTableChange(InternalSchema newSchema, HoodieTableMetaClient m
       throw new HoodieCommitException("Failed to commit " + instantTime + " unable to save inflight metadata ", io);
     }
     Map<String, String> extraMeta = new HashMap<>();
-    extraMeta.put(SerDeHelper.LATEST_SCHEMA, SerDeHelper.toJson(newSchema.setSchemaId(Long.getLong(instantTime))));
+    extraMeta.put(SerDeHelper.LATEST_SCHEMA, SerDeHelper.toJson(newSchema.setSchemaId(Long.parseLong(instantTime))));
     // try to save history schemas
     FileBasedInternalSchemaStorageManager schemasManager = new FileBasedInternalSchemaStorageManager(metaClient);
     schemasManager.persistHistorySchemaStr(instantTime, SerDeHelper.inheritSchemas(newSchema, historySchemaStr));

Original file line number	Diff line number	Diff line change
`@@ -1694,7 +1694,7 @@ private void commitTableChange(InternalSchema newSchema, HoodieTableMetaClient m`
`1694`	`1694`	`throw new HoodieCommitException("Failed to commit " + instantTime + " unable to save inflight metadata ", io);`
`1695`	`1695`	`}`
`1696`	`1696`	`Map<String, String> extraMeta = new HashMap<>();`
`1697`		`- extraMeta.put(SerDeHelper.LATEST_SCHEMA, SerDeHelper.toJson(newSchema.setSchemaId(Long.getLong(instantTime))));`
	`1697`	`+ extraMeta.put(SerDeHelper.LATEST_SCHEMA, SerDeHelper.toJson(newSchema.setSchemaId(Long.parseLong(instantTime))));`
`1698`	`1698`	`// try to save history schemas`
`1699`	`1699`	`FileBasedInternalSchemaStorageManager schemasManager = new FileBasedInternalSchemaStorageManager(metaClient);`
`1700`	`1700`	`schemasManager.persistHistorySchemaStr(instantTime, SerDeHelper.inheritSchemas(newSchema, historySchemaStr));`