This is an automated email from the ASF dual-hosted git repository.

dayangshi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/amoro.git


The following commit(s) were added to refs/heads/master by this push:
     new eabf74ede [Hotfix] Fix wrong usage of Preconditions (#2829)
eabf74ede is described below

commit eabf74ede7ff444a8e1d7b855e885a40f00a96d8
Author: Paul Lin <[email protected]>
AuthorDate: Sun May 12 09:48:34 2024 +0800

    [Hotfix] Fix wrong usage of Preconditions (#2829)
    
    Fix wrong usage of Preconditions
---
 .../src/main/java/org/apache/amoro/server/table/TableRuntime.java   | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/amoro-ams/amoro-ams-server/src/main/java/org/apache/amoro/server/table/TableRuntime.java
 
b/amoro-ams/amoro-ams-server/src/main/java/org/apache/amoro/server/table/TableRuntime.java
index e735e3ac7..3a889d016 100644
--- 
a/amoro-ams/amoro-ams-server/src/main/java/org/apache/amoro/server/table/TableRuntime.java
+++ 
b/amoro-ams/amoro-ams-server/src/main/java/org/apache/amoro/server/table/TableRuntime.java
@@ -101,7 +101,8 @@ public class TableRuntime extends StatedPersistentBase {
       ServerTableIdentifier tableIdentifier,
       TableRuntimeHandler tableHandler,
       Map<String, String> properties) {
-    Preconditions.checkNotNull(tableIdentifier, tableHandler);
+    Preconditions.checkNotNull(tableIdentifier, "ServerTableIdentifier must 
not be null.");
+    Preconditions.checkNotNull(tableHandler, "TableRuntimeHandler must not be 
null.");
     this.tableHandler = tableHandler;
     this.tableIdentifier = tableIdentifier;
     this.tableConfiguration = TableConfiguration.parseConfig(properties);
@@ -111,7 +112,8 @@ public class TableRuntime extends StatedPersistentBase {
   }
 
   protected TableRuntime(TableRuntimeMeta tableRuntimeMeta, 
TableRuntimeHandler tableHandler) {
-    Preconditions.checkNotNull(tableRuntimeMeta, tableHandler);
+    Preconditions.checkNotNull(tableRuntimeMeta, "TableRuntimeMeta must not be 
null.");
+    Preconditions.checkNotNull(tableHandler, "TableRuntimeHandler must not be 
null.");
     this.tableHandler = tableHandler;
     this.tableIdentifier =
         ServerTableIdentifier.of(

Reply via email to