Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19111
**[Test build #81355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81355/testReport)**
for PR 19111 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19111
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
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19018
#19111
---
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 user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19111
**[Test build #81355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81355/testReport)**
for PR 19111 at commit
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/19111
[SPARK-21801][SPARKR][TEST][WIP] set random seed for predictable test
## What changes were proposed in this pull request?
set.seed() before running tests
## How was this patch
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #81354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81354/testReport)**
for PR 19082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
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
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19082
Jenkins, 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81351/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #81351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81351/testReport)**
for PR 19082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81352/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
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 user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81352 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81352/testReport)**
for PR 19110 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19018
ouch. I think this is because
```
# Test unseen labels
data <- data.frame(clicked = base::sample(c(0, 1), 10, replace = TRUE),
someString = base::sample(c("this", "that"), 10,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18869#discussion_r136709446
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/PredicateSuite.scala
---
@@ -151,29 +151,63 @@ class PredicateSuite extends
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18869
@gatorsmile As I said in
https://github.com/apache/spark/pull/18869#discussion_r136706025, we already
have tests in `ExpressionTypeCheckingSuite` to check the predicates don't
support map type. Do
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18869
Could you also add negative test cases for map?
---
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
Github user demoonism commented on a diff in the pull request:
https://github.com/apache/spark/pull/19109#discussion_r136709158
--- Diff: docs/index.md ---
@@ -24,7 +24,7 @@ source, visit [Building Spark](building-spark.html).
Spark runs on both Windows and UNIX-like
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18869#discussion_r136709144
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/PredicateSuite.scala
---
@@ -151,29 +151,63 @@ class PredicateSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81349/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
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 user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81349 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81349/testReport)**
for PR 18869 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
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 user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18869
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81348/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18869
**[Test build #81348 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81348/testReport)**
for PR 18869 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18865
After looking the change, found another two points we should address.
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136709025
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -114,6 +114,17 @@ class JsonFileFormat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18865
**[Test build #81353 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81353/testReport)**
for PR 18865 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18865
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81353/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18865
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
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18865#discussion_r136708996
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JsonFileFormat.scala
---
@@ -114,6 +114,17 @@ class JsonFileFormat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18865
**[Test build #81353 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81353/testReport)**
for PR 18865 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18865
LGTM
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18865
LGTM
---
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
201 - 234 of 234 matches
Mail list logo