Github user kiszk commented on the issue:

    https://github.com/apache/spark/pull/22068
  
    Thanks. BTW, I found another instance in test, not in doc.
    
    Do we address this in this PR?  Or, do we address in another PR?
    @HyukjinKwon  WDYT ?
    
    ```
    class ParquetCompressionCodecPrecedenceSuite extends ParquetTest with 
SharedSQLContext {
      test("Test `spark.sql.parquet.compression.codec` config") {
        Seq("NONE", "UNCOMPRESSED", "SNAPPY", "GZIP", "LZO").foreach { c =>
          withSQLConf(SQLConf.PARQUET_COMPRESSION.key -> c) {
            val expected = if (c == "NONE") "UNCOMPRESSED" else c
            val option = new ParquetOptions(Map.empty[String, String], 
spark.sessionState.conf)
            assert(option.compressionCodecClassName == expected)
          }
        }
      }
    ```


---

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

Reply via email to