maheshk114 commented on a change in pull request #722: HIVE-21956 : Add the
list of table selected by dump in the dump folder.
URL: https://github.com/apache/hive/pull/722#discussion_r303291544
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/repl/ReplDumpTask.java
##########
@@ -376,13 +405,68 @@ private ReplicationSpec
getNewEventOnlyReplicationSpec(Long eventId) {
return rspec;
}
- Long bootStrapDump(Path dumpRoot, DumpMetaData dmd, Path cmRoot, Hive
hiveDb) throws Exception {
+ private void dumpTableListToDumpLocation(List<String> tableList, Path
dbRoot, String dbName, boolean isIncLoad,
+ HiveConf hiveConf) throws
IOException, LoginException {
+ // Empty list will create an empty file to distinguish it from db level
replication. If no file is there, that means
+ // db level replication. If empty file is there, means no table satisfies
the policy.
+ if (tableList == null) {
+ LOG.debug("Table list file is not created for db level replication.");
+ return;
+ }
+
+ // For incremental dump the table list is dumped in _tables/dbname file
and for bootstrap dump, its dumped in
Review comment:
then one directory has to be created for each db .. i think this looks fine
and it was discussed in the stand up ..to be like this
----------------------------------------------------------------
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]
