[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread zsxwing
Github user zsxwing commented on the issue: https://github.com/apache/spark/pull/19247 Thanks! Merging to master and 2.3. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86288/ Test PASSed. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86288 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86288/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86288 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86288/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread zsxwing
Github user zsxwing commented on the issue: https://github.com/apache/spark/pull/19247 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread xysun
Github user xysun commented on the issue: https://github.com/apache/spark/pull/19247 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86243/ Test FAILed. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86243 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86243/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86243 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86243/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86241 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86241/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86241/ Test FAILed. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86241 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86241/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86237/ Test FAILed. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86237 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86237/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread xysun
Github user xysun commented on the issue: https://github.com/apache/spark/pull/19247 Hi @zsxwing I have pushed latest changes (for file sink I'll be honest I simply copied your code =p) I also verified that both tests would fail without the fix. Please review. Thanks. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86237 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86237/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86132/ Test PASSed. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86132 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86132/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86132 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86132/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19247 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86130/ Test FAILed. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86130 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86130/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-14 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19247 **[Test build #86130 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86130/testReport)** for PR 19247 at commit

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-14 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19247 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2018-01-14 Thread xysun
Github user xysun commented on the issue: https://github.com/apache/spark/pull/19247 hi @HyukjinKwon @zsxwing @mgaido91 i have updated code according to the comments, also merged with latest master. Please review. Thanks. ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2017-12-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2017-09-18 Thread xysun
Github user xysun commented on the issue: https://github.com/apache/spark/pull/19247 @joseph-torres @brkyvz @lw-lin can you please take a look? (sorry for uninvited mentions but i just took the latest commits on `FileStreamSource`) ---

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2017-09-15 Thread xysun
Github user xysun commented on the issue: https://github.com/apache/spark/pull/19247 To handle file names with special characters, we should use `URI.getPath` to get decoded path, instead of using `toString`, which may contain other characters different from original path.

[GitHub] spark issue #19247: [Spark-21996][SQL] read files with space in name for str...

2017-09-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19247 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional