GitHub user sylvinus opened a pull request:

    https://github.com/apache/spark/pull/14488

    [SPARK-16826][SQL] Switch to java.net.URI for parse_url()

    ## What changes were proposed in this pull request?
    See the rationale there https://issues.apache.org/jira/browse/SPARK-16826
    
    ## How was this patch tested?
    I've kept the old URL code commented for now, so that people can verify 
that the new unit tests do pass with java.net.URL.
    
    Thanks to @srowen for the help!

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/sylvinus/spark master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/14488.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #14488
    
----
commit a30ca9f4cfde295a811cbe144d6cf165be1227c2
Author: Sylvain Zimmer <[email protected]>
Date:   2016-07-26T21:47:26Z

    [SPARK-16740][SQL] Fix Long overflow in LongToUnsafeRowMap

commit f35b7f663c0e8c68ae62712cf8bef5c3769934a8
Author: Sylvain Zimmer <[email protected]>
Date:   2016-07-26T22:27:51Z

    Add a comment about [SPARK-16740], as suggested by @srowen

commit 98119a08368b1cd1faf3f25a32910ad6717c5c02
Author: Sylvain Zimmer <[email protected]>
Date:   2016-08-02T02:38:37Z

    Investigate switching to java.net.URI for SPARK-16826

commit 06d68e0092edf8730fcdc71de4ec2bcaeccd9e7d
Author: Sylvain Zimmer <[email protected]>
Date:   2016-08-04T00:06:10Z

    Add new tests for PARSE_URL() and fully switch to java.net.URI

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to