anuragshekhar2020 commented on a change in pull request #2029:
URL: https://github.com/apache/hive/pull/2029#discussion_r589154569
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/repl/ReplLoadTask.java
##########
@@ -435,7 +435,7 @@ private TaskTracker addLoadFunctionTasks(Context
loadContext, BootstrapEventsIte
// texts using new DB name. Currently it refers to the source database
name.
}
- CreateViewDesc desc = new CreateViewDesc(dbDotView, table.getAllCols(),
null, table.getParameters(),
+ CreateViewDesc desc = new CreateViewDesc(dbDotView, table.getCols(), null,
table.getParameters(),
Review comment:
I have listed the steps to reproduce the issue in jira.
We do need getPartCols and its part of the constructor parameters for
CreateViewDesc (its in next line so its not showing in the diff, expanded view
will show). However list from getAllCols adds partition columns to both column
list and partition column list and latter in the code flow the check fails due
to duplicate entry.
----------------------------------------------------------------
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]
