simhadri-g commented on code in PR #3307:
URL: https://github.com/apache/hive/pull/3307#discussion_r894630680


##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -4667,6 +4667,9 @@ public static enum ConfVars {
     HIVE_ACID_DIRECT_INSERT_ENABLED("hive.acid.direct.insert.enabled", true,
         "Enable writing the data files directly to the table's final 
destination instead of the staging directory."
         + "This optimization only applies on INSERT operations on ACID 
tables."),
+    HIVE_ACID_NO_RENAME_CTAS_ENABLED("hive.acid.no.rename.ctas.enabled", false,

Review Comment:
   Changed the conf  name to  hive.acid.check.for.concurrent.ctas.enabled as it 
is better suited here.
   
   Direct CTAS patch does not introduce a new conf. It depends on 
hive.acid.direct.insert.enabled .



-- 
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]

Reply via email to