sam-an-cloudera commented on a change in pull request #1133:
URL: https://github.com/apache/hive/pull/1133#discussion_r441826009



##########
File path: 
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreUtils.java
##########
@@ -105,6 +105,7 @@ protected DateFormat initialValue() {
   public static final String DB_EMPTY_MARKER = "!";
 
   public static final String EXTERNAL_TABLE_PURGE = "external.table.purge";
+  public static final String EXTERNAL_TABLE_AUTODELETE = 
"external.table.autodelete";

Review comment:
       After discussion with @nrg4878 and @thejasmn , we thought it's best that 
we don't add this new option because "external.table.purge" is widely used. 
Introducing the new autodelete might cause confusion. 




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

Reply via email to