kasakrisz commented on code in PR #2855:
URL: https://github.com/apache/hive/pull/2855#discussion_r885532179
##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -3381,11 +3381,19 @@ public static enum ConfVars {
MERGE_CARDINALITY_VIOLATION_CHECK("hive.merge.cardinality.check", true,
"Set to true to ensure that each SQL Merge statement ensures that for
each row in the target\n" +
"table there is at most 1 matching row in the source table per SQL
Specification."),
+ SPLIT_UPDATE("hive.split.update", true,
+ "If true, SQL Update statement will be rewritten to a multi-insert
with 2 branches:\n" +
+ "representing delete of existing row and an insert of the
new version of the row.\n" +
+ "Similarly Merge statement will handle WHEN MATCHED UPDATE
by splitting it into 2\n" +
+ "branches of a multi-insert. Updating bucketing and
partitioning columns should\n" +
+ "only be permitted if this is true."),
+ @Deprecated
MERGE_SPLIT_UPDATE("hive.merge.split.update", true,
"If true, SQL Merge statement will handle WHEN MATCHED UPDATE by
splitting it into 2\n" +
"branches of a multi-insert, representing delete of existing row
and an insert of\n" +
"the new version of the row. Updating bucketing and partitioning
columns should\n" +
- "only be permitted if this is true."),
+ "only be permitted if this is true.\n" +
+ "Use hive.split.update instead."),
Review Comment:
Added javadocs, and the `deprecated` string into the comment.
--
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]
