Skip to content

Commit 6de89cd

Browse files
nsivabalansatishkotha
authored andcommitted
[MINOR] Fixing broken test (apache#7123)
1 parent 0a3ad91 commit 6de89cd

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

hudi-sync/hudi-sync-common/src/main/java/org/apache/hudi/sync/common/util/SyncUtilHelpers.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,12 @@ static HoodieSyncTool instantiateMetaSyncTool(String syncToolClassName,
7171
properties.putAll(props);
7272
properties.put(HoodieSyncConfig.META_SYNC_BASE_PATH.key(), targetBasePath);
7373
properties.put(HoodieSyncConfig.META_SYNC_BASE_FILE_FORMAT.key(), baseFileFormat);
74-
String tableName = properties.getString(HoodieSyncConfig.META_SYNC_TABLE_NAME.key());
75-
if (!tableName.equals(tableName.toLowerCase())) {
76-
LOG.warn("Table name \"" + tableName + "\" contains capital letters. Your metastore may automatically convert this to lower case and can cause table not found errors during subsequent syncs.");
74+
if (properties.containsKey(HoodieSyncConfig.META_SYNC_TABLE_NAME.key())) {
75+
String tableName = properties.getString(HoodieSyncConfig.META_SYNC_TABLE_NAME.key());
76+
if (!tableName.equals(tableName.toLowerCase())) {
77+
LOG.warn(
78+
"Table name \"" + tableName + "\" contains capital letters. Your metastore may automatically convert this to lower case and can cause table not found errors during subsequent syncs.");
79+
}
7780
}
7881

7982
if (ReflectionUtils.hasConstructor(syncToolClassName,

0 commit comments

Comments
 (0)