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

    https://github.com/apache/spark/pull/23108#discussion_r239361792
  
    --- Diff: 
sql/hive/src/test/scala/org/apache/spark/sql/hive/orc/HiveOrcSourceSuite.scala 
---
    @@ -73,9 +73,11 @@ class HiveOrcSourceSuite extends OrcSuite with 
TestHiveSingleton {
           sql(
             s"""
               |CREATE TABLE $tableName
    +
               |USING org.apache.spark.sql.hive.orc
               |OPTIONS (
    -          |  PATH '${new File(orcTableAsDir.getAbsolutePath).toURI}'
    +          |  PATH '${new File(orcTableAsDir.getAbsolutePath
    +        ).toURI}'
    --- End diff --
    
    The above change in line 76 ~ 80 looks strange and irrelevant. Let's revert 
this change.


---

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

Reply via email to