kasakrisz commented on code in PR #4832:
URL: https://github.com/apache/hive/pull/4832#discussion_r1391213635
##########
ql/src/java/org/apache/hadoop/hive/ql/Context.java:
##########
@@ -200,6 +201,11 @@ public void setOperation(Operation operation) {
this.operation = operation;
}
+ public void setOperation(Operation operation, boolean splitUpdate) {
Review Comment:
Copy-on-write and "default" is the same from operation standpoint. I would
introduce enum only when it is necessary. I was also thinking about treating
split update as merge in a follow-up patch.
--
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]
