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

    https://github.com/apache/spark/pull/21902#discussion_r205937278
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
    @@ -1449,6 +1451,16 @@ object SQLConf {
         .intConf
         .checkValues((1 to 9).toSet + Deflater.DEFAULT_COMPRESSION)
         .createWithDefault(Deflater.DEFAULT_COMPRESSION)
    +
    +  val AVRO_XZ_LEVEL = buildConf("spark.sql.avro.xz.level")
    --- End diff --
    
    > If you faced a case that this configuration is useful, im okay but if 
not, I would suggest to add it later when it's needed.
    
    The config is not so useful as `LZMA2` codec itself. Its default value 
covers the use case. I will remove the config from the PR. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to