[ 
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)

Reply via email to