Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16894
**[Test build #72734 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72734/consoleFull)**
for PR 16894 at commit
[`11d2684`](https://github.com/apache/spark/commit/1
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/16894
[SPARK-17897] [SQL] [BACKPORT-2.0] Fixed IsNotNull Constraint Inference Rule
### What changes were proposed in this pull request?
This PR is to backport https://github.com/apache/spark/p
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16893
**[Test build #72733 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72733/testReport)**
for PR 16893 at commit
[`e68eab0`](https://github.com/apache/spark/commit/e6
GitHub user lins05 opened a pull request:
https://github.com/apache/spark/pull/16893
[SPARK-19555][SQL] Improve the performance of StringUtils.escapeLikeRegex
method
## What changes were proposed in this pull request?
Copied from [SPARK-19555](https://issues.apache.org/jira
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16891
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72731/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16891
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16891
**[Test build #72731 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72731/testReport)**
for PR 16891 at commit
[`41d3362`](https://github.com/apache/spark/commit/4
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16067
Sure, let me back port it now.
---
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 a
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16620
Also, if you implement the new change I proposed, I think it's relatively
straightforward to write a new test in DAGSchedulerSuite for the new behavior
(which will be pretty similar to the tes
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16892
cc @mateiz, whose test I deleted / rolled into the existing one
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/16876
---
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 ena
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16876
Thanks all for the review; merged this into 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
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16876
@jinxing64 I wrote a long comment on your other PR to attempt to explain
(my current understanding of) these issues!
---
If your project is set up for it, you can reply to this email and have
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16620
tl;dr I donât think Markâs change is quite correct, which is why the
tests were failing. Instead, I think we need to replace the failedEpoch
if/else statement and the pendingPartitions up
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16892
**[Test build #72732 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72732/testReport)**
for PR 16892 at commit
[`06b26d3`](https://github.com/apache/spark/commit/06
GitHub user kayousterhout opened a pull request:
https://github.com/apache/spark/pull/16892
[SPARK-19560] Improve DAGScheduler tests.
This commit improves the tests that check the case when a
ShuffleMapTask completes successfully on an executor that has
failed. This commit i
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16785
@cloud-fan yeah, i agreed with you and @hvanhovell.
For too slow constraint propagation, except for attacking
`getAliasedConstraints` like this change, maybe we can have other way to
improve
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16724
@cloud-fan Is the current change not suitable? We can change it to only
preserve data order when specifying partitioning and no bucketing for the
output.
This change only adds a new construc
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r100660551
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,623 @@
+/*
+
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16228#discussion_r100660535
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/Range.scala
---
@@ -0,0 +1,117 @@
+/*
+ * Licensed
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16228#discussion_r100660524
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/JoinEstimation.scala
---
@@ -0,0 +1,314 @@
+/*
+ *
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16228#discussion_r100660451
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/JoinEstimation.scala
---
@@ -0,0 +1,314 @@
+/*
+ *
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16228#discussion_r100660417
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/JoinEstimation.scala
---
@@ -0,0 +1,314 @@
+/*
+ *
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16228#discussion_r100660406
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -340,14 +340,22 @@ case class Join(
Github user sureshthalamati commented on the issue:
https://github.com/apache/spark/pull/16847
@cloud-fan @gatorsmile
I created two PR for the same issue , as I was not sure what is the best
approach to fix this issue. I would appreciate your feedback. Thanks
---
If your projec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16891
**[Test build #72731 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72731/testReport)**
for PR 16891 at commit
[`41d3362`](https://github.com/apache/spark/commit/41
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16715
Could you please add tag "[PYTHON]" to the PR title?
Also, please remove "Please review
http://spark.apache.org/contributing.html before opening a pull request." from
the PR description since
GitHub user sureshthalamati opened a pull request:
https://github.com/apache/spark/pull/16891
[SPARK-19318][SQL] Fix to treat JDBC connection properties specified by the
user in case-sensitive manner.
## What changes were proposed in this pull request?
The reason for test failur
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16870
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72730/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16870
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16870
**[Test build #72730 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72730/testReport)**
for PR 16870 at commit
[`2dc241e`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16847
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72728/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16847
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16847
**[Test build #72728 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72728/testReport)**
for PR 16847 at commit
[`81e9060`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16386
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72729/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16386
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16386
**[Test build #72729 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72729/testReport)**
for PR 16386 at commit
[`b0a5cc8`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72727/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16826
**[Test build #72727 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72727/testReport)**
for PR 16826 at commit
[`6da6bda`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16871
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72724/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16871
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16871
**[Test build #72724 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72724/testReport)**
for PR 16871 at commit
[`7f8a2cb`](https://github.com/apache/spark/commit/7
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16889
FYI the test that failed here seems to be flaky and probably not related to
this PR: https://issues.apache.org/jira/browse/SPARK-19559
---
If your project is set up for it, you can reply to t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16870
**[Test build #72730 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72730/testReport)**
for PR 16870 at commit
[`2dc241e`](https://github.com/apache/spark/commit/2d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15009
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72723/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15009
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
e
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/16870#discussion_r100656533
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -465,15 +465,15 @@ case class DateFormat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #72723 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72723/testReport)**
for PR 15009 at commit
[`cc2c0be`](https://github.com/apache/spark/commit/c
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16686
I recently have noticed a few flaky test failures of
KafkaSourceSuite.subscribing topic by pattern with topic deletions (e.g.,
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilde
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16890
Can one of the admins verify this patch?
---
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 feat
GitHub user muyannian opened a pull request:
https://github.com/apache/spark/pull/16890
when colum is use alias ,the order by result is wrong
i write two sql.
the first order by result is wrong, but the second order by result is
right,that may be a bug?
---sql 1
s
Github user erenavsarogullari commented on the issue:
https://github.com/apache/spark/pull/15604
build failure looks unrelated (due to timeout at KafkaSourceSuite level)
Jenkins, retest this please
---
If your project is set up for it, you can reply to this email and have you
Github user ouyangxiaochen commented on the issue:
https://github.com/apache/spark/pull/16868
I have run test cases successfully. Please run the test cases again.Thanks
a lot! @SparkQA
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15604
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15604
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72725/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15604
**[Test build #72725 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72725/testReport)**
for PR 15604 at commit
[`f84abe7`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16386
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16386
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72726/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16386
**[Test build #72726 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72726/testReport)**
for PR 16386 at commit
[`0f4686d`](https://github.com/apache/spark/commit/0
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16777
Let me check other DBMSs and back.
---
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
enab
Github user NathanHowell commented on the issue:
https://github.com/apache/spark/pull/16386
@cloud-fan I just pushed a few more changes to address some of your
comments. I'll be back later next week to continue work.
---
If your project is set up for it, you can reply to this email a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16386
**[Test build #72729 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72729/testReport)**
for PR 16386 at commit
[`b0a5cc8`](https://github.com/apache/spark/commit/b0
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100653879
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100653835
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100653757
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100653580
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16880
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72721/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16880
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16880
**[Test build #72721 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72721/testReport)**
for PR 16880 at commit
[`cae981f`](https://github.com/apache/spark/commit/c
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100652445
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonDataSource.scala
---
@@ -0,0 +1,213 @@
+/*
+ * Licensed
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16847
**[Test build #72728 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72728/testReport)**
for PR 16847 at commit
[`81e9060`](https://github.com/apache/spark/commit/81
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/16847
retest this please
---
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 s
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100652259
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100652192
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100651990
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100651910
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100651706
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonInferSchema.scala
---
@@ -79,7 +80,7 @@ private[sql] object Json
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100651385
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100651178
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100651103
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100651055
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100650756
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100650577
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100650500
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100650450
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -394,36 +447,32 @@ class JacksonParser(
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16889
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16889
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72720/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16889
**[Test build #72720 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72720/testReport)**
for PR 16889 at commit
[`ac09ad5`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100650024
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,123 @@ class JsonSuite extends Que
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100649836
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -48,69 +47,110 @@ class JacksonParser(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100649847
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonInferSchema.scala
---
@@ -79,7 +80,7 @@ private[sql] object JsonInf
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100649748
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -97,46 +91,13 @@ class JsonFileFormat exten
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100649635
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -31,10 +31,17 @@ import
org.apache.spark.sql.cataly
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/11887
@HyukjinKwon probably. Let me ask Michael. @mgummelt what do you think?
what options do we have?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/16161
Thanks. Backported to branch-2.1.
---
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100649228
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -394,36 +447,32 @@ class JacksonParser(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100649101
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -394,36 +447,32 @@ class JacksonParser(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100648863
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -394,36 +447,32 @@ class JacksonParser(
}
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r100648524
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -394,36 +447,32 @@ class JacksonParser(
}
1 - 100 of 435 matches
Mail list logo