zhangbutao commented on code in PR #6088:
URL: https://github.com/apache/hive/pull/6088#discussion_r2465770546
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/repl/ReplLoadTask.java:
##########
@@ -448,8 +449,9 @@ private void setDbReadOnly() {
Map<String, String> props = new HashMap<>();
props.put(READONLY, Boolean.TRUE.toString());
+ // TODO catalog. Need to double check the actual catalog here. Depend on
HIVE-29278
Review Comment:
You are right. Some task can only do in NATIVE DBs. Like this ReploadTask.
This TODO is added to verify if the correct native catalog name is obtained.
BTW, besides the default `hive ` native catalog, we still need to support
other multiple native catalogs.
As for external catalogs, we need to skip those tasks that are exclusive to
native catalogs. However, the specific types of external catalogs (e.g., JDBC,
REST) have not yet been defined, and this part of the work can be refined and
completed subsequently. HIVE-29243 can be used to track this matter.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
