Github user mengxr commented on the issue:

    https://github.com/apache/spark/pull/22449
  
    @WeichenXu123 I think we should fix the test instead of removing "//" from 
URI if authority is empty. Because both "scheme:/" and "scheme:///" are valid.
    
    ~~~scala
    scala> val u1 = new URI("file:///a/b/c")
    u1: java.net.URI = file:///a/b/c
    
    scala> val u2 = new URI("file:/a/b/c")
    u2: java.net.URI = file:/a/b/c
    
    scala> u1 == u2
    res1: Boolean = true
    ~~~
    
    Shall we update the test? Instead of compare the row record, we compare its 
fields one by one and convert `origin` to `URI` before comparison?


---

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

Reply via email to