Github user mgaido91 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22078#discussion_r209957030
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
    @@ -626,6 +626,14 @@ object SQLConf {
         .stringConf
         .createWithDefault("parquet")
     
    +  val DATA_SOURCE_TABLE_INHERIT_PERMS = 
buildConf("spark.sql.datasource.table.inherit.perms")
    +    .internal()
    +    .doc("Set this to true if the table directories should be inheriting 
the permission " +
    +      "of the warehouse or database directory " +
    --- End diff --
    
    yes, but that does a different thing IIUC, ie. that copies the permissions 
from the warehouse dir for the new tables created there. Here we are not doing 
the same, we are copying the permissions from the previous table dir to the new 
table dir after the overwrite.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to