You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2020/01/16 04:14:01 UTC

[spark] branch branch-2.4 updated: [SPARK-30312][SQL][FOLLOWUP] Rename conf by adding `.enabled`

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-2.4
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-2.4 by this push:
     new 830a4ec  [SPARK-30312][SQL][FOLLOWUP] Rename conf by adding `.enabled`
830a4ec is described below

commit 830a4ec59b86253f18eb7dfd6ed0bbe0d7920e5b
Author: Liang-Chi Hsieh <vi...@gmail.com>
AuthorDate: Wed Jan 15 20:09:12 2020 -0800

    [SPARK-30312][SQL][FOLLOWUP] Rename conf by adding `.enabled`
    
    Based on the [comment](https://github.com/apache/spark/pull/26956#discussion_r366680558), this patch changes the SQL config name from `spark.sql.truncateTable.ignorePermissionAcl` to `spark.sql.truncateTable.ignorePermissionAcl.enabled`.
    
    Make this config consistent other SQL configs.
    
    No.
    
    Unit test.
    
    Closes #27210 from viirya/truncate-table-permission-followup.
    
    Authored-by: Liang-Chi Hsieh <vi...@gmail.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
    (cherry picked from commit be4d825872b41e04e190066e550217362b82061e)
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
index c326b27..760a9db 100644
--- a/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
+++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
@@ -1578,7 +1578,7 @@ object SQLConf {
       .createWithDefault(false)
 
   val TRUNCATE_TABLE_IGNORE_PERMISSION_ACL =
-    buildConf("spark.sql.truncateTable.ignorePermissionAcl")
+    buildConf("spark.sql.truncateTable.ignorePermissionAcl.enabled")
       .internal()
       .doc("When set to true, TRUNCATE TABLE command will not try to set back original " +
         "permission and ACLs when re-creating the table/partition paths.")


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org