You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2022/01/02 01:47:54 UTC

[GitHub] [hudi] XuQianJin-Stars commented on a change in pull request #4486: [HUDI-3132] Minor fixes for HoodieCatalog

XuQianJin-Stars commented on a change in pull request #4486:
URL: https://github.com/apache/hudi/pull/4486#discussion_r777155692



##########
File path: hudi-flink/src/main/java/org/apache/hudi/table/catalog/HoodieCatalog.java
##########
@@ -145,7 +145,7 @@ public CatalogDatabase getDatabase(String databaseName) throws DatabaseNotExistE
 
   @Override
   public boolean databaseExists(String databaseName) throws CatalogException {
-    checkArgument(!StringUtils.isNullOrWhitespaceOnly(databaseName));
+    checkArgument(!StringUtils.isNullOrEmpty(databaseName));

Review comment:
       `isNullOrWhitespace` includes the function of `isNullOrEmpty`. 




-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org