You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by dongjoon-hyun <gi...@git.apache.org> on 2018/05/06 02:03:50 UTC

[GitHub] spark pull request #21070: [SPARK-23972][BUILD][SQL] Update Parquet to 1.10....

Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21070#discussion_r186276440
  
    --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
    @@ -345,7 +345,7 @@ object SQLConf {
           "snappy, gzip, lzo.")
         .stringConf
         .transform(_.toLowerCase(Locale.ROOT))
    -    .checkValues(Set("none", "uncompressed", "snappy", "gzip", "lzo"))
    +    .checkValues(Set("none", "uncompressed", "snappy", "gzip", "lzo", "lz4", "brotli", "zstd"))
    --- End diff --
    
    Could you update [sql-programming-guide.md](https://github.com/apache/spark/blame/master/docs/sql-programming-guide.md#L967) together?


---

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