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

    https://github.com/apache/spark/pull/20504#discussion_r166133771
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
    @@ -1180,6 +1180,19 @@ object SQLConf {
           .booleanConf
           .createWithDefault(true)
     
    +  object TypeCoercionMode extends Enumeration {
    +    val default, hive = Value
    --- End diff --
    
    Hi, @wangyum .
    What about using `val native` instead of `val default` here?
    Then, I think you can remove unnecessary renaming like 
`~/native/binaryComparison.sql.out → ~/default/binaryComparison.sql.out`?


---

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

Reply via email to