pvary commented on code in PR #3346:
URL: https://github.com/apache/hive/pull/3346#discussion_r892269558
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/repl/ReplLoadTask.java:
##########
@@ -703,6 +705,7 @@ private void createBuilderTask(List<Task<?>> rootTasks) {
private int executeIncrementalLoad(long loadStartTime) throws Exception {
// If replication policy is changed between previous and current repl
load, then drop the tables
// that are excluded in the new replication policy.
+ this.childTasks = (this.childTasks == null) ? new ArrayList<>() :
this.childTasks;
Review Comment:
How much of this refactor needed for the specific change?
We might want to do it in a separate PR so we can backport the fix without
the conflicts generated by the refactor when needed
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
