[ https://issues.apache.org/jira/browse/HIVE-19476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16469252#comment-16469252 ]
Sankar Hariappan commented on HIVE-19476: ----------------------------------------- [~sershe], Thanks for the review! copyFiles path is for Export command while exportAsFilesList is for replication. Currently, in replication path, caller will make the valid data paths (for MM/Full ACID/Non-ACID tables) and invoke FileOperations.export. Export path for non-ACID table also pass valid data path but for MM tables we make the valid data paths within FileOperations class and also set different destPath for each path. So, it would make sense to move this valid paths generation code to the caller and also invoke FileOperations.export for each valid path with appropriate destPath as inputs. If you think, it make sense, will take it up in a follow-up jira. > Fix failures in TestReplicationScenariosAcidTables, > TestReplicationOnHDFSEncryptedZones and TestCopyUtils > --------------------------------------------------------------------------------------------------------- > > Key: HIVE-19476 > URL: https://issues.apache.org/jira/browse/HIVE-19476 > Project: Hive > Issue Type: Bug > Components: HiveServer2, repl > Affects Versions: 3.0.0 > Reporter: Sankar Hariappan > Assignee: Sankar Hariappan > Priority: Major > Labels: Tests, pull-request-available > Fix For: 3.0.0, 3.1.0 > > Attachments: HIVE-19476.01.patch > > > TestReplicationScenariosAcidTables broken due to HIVE-17657. > cc [~sershe] > TestReplicationOnHDFSEncryptedZones and TestCopyUtils are failing due to > auto-stats gathered which thrown exception irrelevant to theses tests. > > -- This message was sent by Atlassian JIRA (v7.6.3#76005)