woofyzhao commented on code in PR #4041:
URL: https://github.com/apache/incubator-inlong/pull/4041#discussion_r862641758
##########
inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/sort/util/SinkInfoUtils.java:
##########
@@ -182,13 +183,15 @@ private static HiveSinkInfo
createHiveSinkInfo(HiveSinkResponse hiveInfo, List<F
}).collect(Collectors.toList());
}
- // dataPath = dataPath + / + tableName
- StringBuilder dataPathBuilder = new StringBuilder();
+ // dataPath = dataPath + / + dbName + / + tableName
Review Comment:
Fixed compatibility issue for the default hive db.
No matter where you create hive table the table name must be part of the
path and must also be specified here to sure sort writing to the correct file.
The default hive location is fine. All that's left is to take the default
database special case into account.
--
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]
