maheshk114 commented on a change in pull request #945: HIVE-22998 Dump
partition info if hive.repl.dump.metadata.only.for.ex…
URL: https://github.com/apache/hive/pull/945#discussion_r391378029
##########
File path:
ql/src/java/org/apache/hadoop/hive/ql/parse/repl/dump/events/AbstractEventHandler.java
##########
@@ -94,7 +94,8 @@ protected void writeFileEntry(String dbName, Table table,
String file, BufferedW
throws IOException, LoginException, MetaException,
HiveFatalException {
HiveConf hiveConf = withinContext.hiveConf;
String distCpDoAsUser =
hiveConf.getVar(HiveConf.ConfVars.HIVE_DISTCP_DOAS_USER);
- if (!Utils.shouldDumpMetaDataOnly(table, withinContext.hiveConf)) {
+ if (!Utils.shouldDumpMetaDataOnlyForExternalTables(table,
withinContext.hiveConf)
Review comment:
for external table we don't dump data related events ..so i don't think we
need to add this check
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]