Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9211
---
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-152523343
I'm going to merge this if there are no objections, to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user zjffdu commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151407652
@rxin @srowen will add unit test soon
---
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151406183
@zjffdu are you working on this?
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151415652
**[Test build #44416 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44416/consoleFull)**
for PR 9211 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151414326
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151414301
Merged build triggered.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151446645
**[Test build #44422 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44422/consoleFull)**
for PR 9211 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151420607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151420603
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151420480
**[Test build #44416 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44416/consoleFull)**
for PR 9211 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151446002
Merged build triggered.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151446020
Merged build started.
---
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 user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/9211#discussion_r43116597
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -871,7 +869,6 @@ class JsonSuite extends QueryTest
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151477972
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151477977
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/9211#discussion_r43115351
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -871,7 +869,6 @@ class JsonSuite extends QueryTest
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9211#discussion_r43116120
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -871,7 +869,6 @@ class JsonSuite extends QueryTest
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-151477759
**[Test build #44422 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44422/consoleFull)**
for PR 9211 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/9211#discussion_r43123000
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -871,7 +869,6 @@ class JsonSuite extends QueryTest
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9211#discussion_r43122465
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -871,7 +869,6 @@ class JsonSuite extends QueryTest
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9211#discussion_r43123523
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -871,7 +869,6 @@ class JsonSuite extends QueryTest
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-150104148
Can you add a unit test for this?
---
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 user zjffdu opened a pull request:
https://github.com/apache/spark/pull/9211
[SPARK-11226] [SQL] Empty line in json file should be skipped
Currently the empty line in json file will be parsed into Row with all null
field values. But in json, "{}" represents a json object,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-150086816
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-150086790
Merged build triggered.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-150087054
**[Test build #44121 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44121/consoleFull)**
for PR 9211 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-150087486
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-150087483
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9211#issuecomment-150087475
**[Test build #44121 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44121/consoleFull)**
for PR 9211 at commit
30 matches
Mail list logo