Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67433/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15597
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/15597
**[Test build #67433 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67433/consoleFull)**
for PR 15597 at commit
[`32e0b36`](https://github.com/apache/spark/commit/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15053
@srowen Do you mind if I ask to trigger the build via Jenkins if it looks
reasonable (or maybe "ok to test")? Let me try to take a look closer but I
would like @holdenk also to take a look pleas
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15608
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67440/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15608
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/15608
**[Test build #67440 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67440/consoleFull)**
for PR 15608 at commit
[`e6afa4b`](https://github.com/apache/spark/commit/
Github user mortada commented on the issue:
https://github.com/apache/spark/pull/15053
@HyukjinKwon I've created a JIRA ticket and also went through all the files
you mentioned, please take a look
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15609
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
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 priyankagargnitk opened a pull request:
https://github.com/apache/spark/pull/15609
[SPARK-18048][SQL] To make behaviour of If consistent, in case of true
expression and false expression are of compatible data types.
## What changes were proposed in this pull request?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67429/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #67429 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67429/consoleFull)**
for PR 15428 at commit
[`70cee57`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15428
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67430/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #67430 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67430/consoleFull)**
for PR 15428 at commit
[`b14fbab`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67443 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67443/consoleFull)**
for PR 15607 at commit
[`0b9fa4f`](https://github.com/apache/spark/commit/0
Github user tejasapatil closed the pull request at:
https://github.com/apache/spark/pull/15605
---
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 i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15541
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67428/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15541
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/15541
**[Test build #67428 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67428/consoleFull)**
for PR 15541 at commit
[`a820e96`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
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/15607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67442/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67442 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67442/consoleFull)**
for PR 15607 at commit
[`b44dada`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67442/consoleFull)**
for PR 15607 at commit
[`b44dada`](https://github.com/apache/spark/commit/b
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15607
Hmm, it works fine with
`roxygen2 "roxygen2" "/usr/local/lib/R/site-library" "5.0.1"`
maybe it is the older version on Jenkins?
```
First time using roxygen2 4.0. Upgr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67441/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
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/15607
**[Test build #67441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67441/consoleFull)**
for PR 15607 at commit
[`4442abe`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67441/consoleFull)**
for PR 15607 at commit
[`4442abe`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15608
**[Test build #67440 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67440/consoleFull)**
for PR 15608 at commit
[`e6afa4b`](https://github.com/apache/spark/commit/e
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15608
cc @felixcheung I recall we talked about this before. I first wanted to
handle all the argument type checking but I just decided to do what I am pretty
sure of (I remember we were concern of swe
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/15608
[SPARK-17838][SparkR] Check named arguments for options and use formatted R
friendly message from JVM exception message
## What changes were proposed in this pull request?
This PR prop
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15566
**[Test build #67439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67439/consoleFull)**
for PR 15566 at commit
[`d4fbeb2`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67438/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67438 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67438/consoleFull)**
for PR 15607 at commit
[`7d6e605`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15607
**[Test build #67438 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67438/consoleFull)**
for PR 15607 at commit
[`7d6e605`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15607
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/15605
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/15605
**[Test build #67431 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67431/consoleFull)**
for PR 15605 at commit
[`dac2f49`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67431/
Test FAILed.
---
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/15607
[SPARK-16137][SPARKR] randomForest for R
## What changes were proposed in this pull request?
Random Forest Regression and Classification for R
Clean-up/reordering generics.R
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15581
@zsxwing shall we backport
[this](https://github.com/apache/spark/pull/14803) first? Seems in 2.0 we don't
support partitioned file source.
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14426
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67425/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14426
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/15606
**[Test build #67437 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67437/consoleFull)**
for PR 15606 at commit
[`ba880d8`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14426
**[Test build #67425 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67425/consoleFull)**
for PR 14426 at commit
[`dfe6a3e`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15606
cc @gatorsmile @yhuai
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15553
**[Test build #67436 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67436/consoleFull)**
for PR 15553 at commit
[`ec49e8b`](https://github.com/apache/spark/commit/e
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/15606
[SPARK-18070][SQL] binary operator should not consider nullability when
comparing input types
## What changes were proposed in this pull request?
Binary operator requires its inputs to b
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/15553
Jenkins, test 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 wishe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15596
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67427/
Test FAILed.
---
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/14136
@rxin @hvanhovell Thanks for your advice, I'll address them ASAP!
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15596
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/15596
**[Test build #67427 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67427/consoleFull)**
for PR 15596 at commit
[`6d7095c`](https://github.com/apache/spark/commit/
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14136
@jiangxb1987 can you add some test cases based on Herman's comment?
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13891
**[Test build #67435 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67435/consoleFull)**
for PR 13891 at commit
[`a6b5a16`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13891
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/13891
**[Test build #67434 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67434/consoleFull)**
for PR 13891 at commit
[`1081e64`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13891
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67434/
Test FAILed.
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15541#discussion_r84621271
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -250,24 +251,24 @@ private[spark] class TaskSchedulerImpl(
priv
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13891
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13891
**[Test build #67432 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67432/consoleFull)**
for PR 13891 at commit
[`1f3ff96`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13891
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67432/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14638
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67426/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14638
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/14638
**[Test build #67426 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67426/consoleFull)**
for PR 14638 at commit
[`9010388`](https://github.com/apache/spark/commit/
Github user zhzhan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15541#discussion_r84621076
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -250,24 +251,24 @@ private[spark] class TaskSchedulerImpl(
priv
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15596#discussion_r84621075
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/BufferedRowIterator.java
---
@@ -39,7 +39,7 @@
protected int partitionIndex = -1;
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15541
One concern I have is the cluster of machines with different codes. It is
possibly if you construct your cluster with solution like ec2 and buy different
kinds of nodes. In this case, the balance ass
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15541
LGTM. I have a question for the legacy code, but not related to this change.
---
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 pr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13891
**[Test build #67434 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67434/consoleFull)**
for PR 13891 at commit
[`1081e64`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15597
**[Test build #67433 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67433/consoleFull)**
for PR 15597 at commit
[`32e0b36`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15605
**[Test build #67431 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67431/consoleFull)**
for PR 15605 at commit
[`dac2f49`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13891
**[Test build #67432 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67432/consoleFull)**
for PR 13891 at commit
[`1f3ff96`](https://github.com/apache/spark/commit/1
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/15605
cc @hvanhovell
---
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,
GitHub user tejasapatil opened a pull request:
https://github.com/apache/spark/pull/15605
[WIP] [SPARK-18067] [SQL] SortMergeJoin adds shuffle if join predicates
have non partitioned columns
## What changes were proposed in this pull request?
See https://issues.apache.org/j
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #67430 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67430/consoleFull)**
for PR 15428 at commit
[`b14fbab`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15428
**[Test build #67429 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67429/consoleFull)**
for PR 15428 at commit
[`70cee57`](https://github.com/apache/spark/commit/7
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/15212
Hi @yanboliang and @srowen , could you please review whether this PR
includes all your comments. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user zhzhan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15541#discussion_r84619879
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -250,24 +251,24 @@ private[spark] class TaskSchedulerImpl(
priv
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15541
**[Test build #67428 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67428/consoleFull)**
for PR 15541 at commit
[`a820e96`](https://github.com/apache/spark/commit/a
Github user zhzhan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15541#discussion_r84619023
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskAssigner.scala
---
@@ -0,0 +1,229 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
Could I please hear your thoughts @rxin, @srowen and @jodersky about
changing the format to
https://github.com/apache/spark/pull/15513#issuecomment-255594355 or
https://github.com/apache/spark/
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15541#discussion_r84618189
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskAssigner.scala
---
@@ -0,0 +1,229 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15596
**[Test build #67427 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67427/consoleFull)**
for PR 15596 at commit
[`6d7095c`](https://github.com/apache/spark/commit/6
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
https://github.com/apache/spark/pull/15513#issuecomment-255632253 I
initially wanted to propose that idea but I didn't becuase I was worried of
ignoring existing name rules in each expression.
Github user Astralidea commented on the issue:
https://github.com/apache/spark/pull/15588
@lw-lin
spark.scheduler.minRegisteredResourcesRatio does not work.
The reason it may could be I use mesos and it run executor not through
driver.
but I still need to make sure it have
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14638
**[Test build #67426 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67426/consoleFull)**
for PR 14638 at commit
[`9010388`](https://github.com/apache/spark/commit/9
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15541
LGTM except a few minor comments. Could @rxin do the final check?
Thank you for your work! @zhzhan
---
If your project is set up for it, you can reply to this email and have your
reply
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15541#discussion_r84616602
--- Diff: docs/configuration.md ---
@@ -1350,6 +1350,20 @@ Apart from these, the following properties are also
available, and may be useful
Shoul
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14426
**[Test build #67425 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67425/consoleFull)**
for PR 14426 at commit
[`dfe6a3e`](https://github.com/apache/spark/commit/d
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15541#discussion_r84616347
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskAssigner.scala
---
@@ -0,0 +1,229 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13891
**[Test build #67424 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67424/consoleFull)**
for PR 13891 at commit
[`513e791`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13891
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/13891
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67424/
Test FAILed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84615985
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/bitwiseExpressions.scala
---
@@ -27,8 +27,16 @@ import org.apache.spark.
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
@gatorsmile Oh, they should be `literal` because in most cases users will
give the input as `literal` for those cases. I though I used a word `literal`
for those foldable expressions.
---
If y
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15513#discussion_r84615684
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CallMethodViaReflection.scala
---
@@ -43,11 +43,20 @@ import org.apache.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13891
**[Test build #67424 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67424/consoleFull)**
for PR 13891 at commit
[`513e791`](https://github.com/apache/spark/commit/5
1 - 100 of 248 matches
Mail list logo