[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-21 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15138 Thanks! --- 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

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-21 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15138 Merged to master --- 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

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-18 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/15138 Yes, you are right and also yes, the purpose of the setting is to prevent OOM

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15138 Merged build finished. Test PASSed. --- 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

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

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

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-18 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15138 **[Test build #65575 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65575/consoleFull)** for PR 15138 at commit

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-18 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/15138 I think you know this area better than anyone. The update seems reasonable. So the only substantive change is to not limit the maximum length of a line? I tend to agree with removing arbitrary

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

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

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

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

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15138 Merged build finished. Test PASSed. --- 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

[GitHub] spark issue #15138: [SPARK-17583][SQL] Remove uesless rowSeparator variable ...

2016-09-18 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15138 **[Test build #65569 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65569/consoleFull)** for PR 15138 at commit