[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13104#issuecomment-219958551 **[Test build #58759 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58759/consoleFull)** for PR 13104 at commit

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread jurriaan
Github user jurriaan commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63660428 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -110,6 +118,10 @@ class TextSuite extends

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63659619 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -110,6 +118,10 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63659571 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread jurriaan
Github user jurriaan commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63658913 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13104#issuecomment-219950378 **[Test build #58757 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58757/consoleFull)** for PR 13104 at commit

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread jurriaan
Github user jurriaan commented on the pull request: https://github.com/apache/spark/pull/13104#issuecomment-219950253 @rxin changed the directory name to text-partitioned --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63655817 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-18 Thread jurriaan
Github user jurriaan commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63655222 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-17 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13104#issuecomment-219900849 It'd be better to change the directory name from partitioned to text-partitioned? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-17 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63632543 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-17 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63584539 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-17 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63584448 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-17 Thread jurriaan
Github user jurriaan commented on the pull request: https://github.com/apache/spark/pull/13104#issuecomment-219653230 @liancheng What do you think? --- 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

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-14 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13104#issuecomment-219207979 Let me cc @liancheng --- 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