aasha commented on a change in pull request #1529:
URL: https://github.com/apache/hive/pull/1529#discussion_r509991386
##########
File path:
ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/load/table/LoadTable.java
##########
@@ -270,10 +269,9 @@ static TableLocationTuple tableLocation(ImportTableDesc
tblDesc, Database parent
Path dataPath = fromURI;
Path tmpPath = tgtPath;
- // if move optimization is enabled, copy the files directly to the target
path. No need to create the staging dir.
+ // if acid tables, copy the files directly to the target path. No need to
create the staging dir.
LoadFileType loadFileType;
- if (replicationSpec.isInReplicationScope() &&
- context.hiveConf.getBoolVar(REPL_ENABLE_MOVE_OPTIMIZATION)) {
+ if (replicationSpec.isInReplicationScope() &&
AcidUtils.isTransactionalTable(table)) {
Review comment:
For tables irrespective of move optimization, move task is always
created and if its transactional tables, move task handles the case
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
