Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22921
**[Test build #98480 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98480/testReport)**
for PR 22921 at commit
Github user arman1371 commented on the issue:
https://github.com/apache/spark/pull/22889
@dongjoon-hyun . yes
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user arman1371 closed the pull request at:
https://github.com/apache/spark/pull/22889
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22889
Thank you for the advice, @gatorsmile and @cloud-fan .
@arman1371 . It's a trade-off issue and the benefit looks smaller than the
benefit. If you don't mind, could you close this PR?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22950
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22950
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22823
Hi, @yucai .
Since https://github.com/apache/spark/pull/22847 is merged, could you
rebase this to the master gain? Thanks!
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22950
**[Test build #98488 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98488/testReport)**
for PR 22950 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22950
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #98487 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98487/testReport)**
for PR 22612 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22612
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #98486 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98486/testReport)**
for PR 22612 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22612
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98486/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #98486 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98486/testReport)**
for PR 22612 at commit
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22612#discussion_r230864147
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -394,9 +394,15 @@ private[spark] object JsonProtocol {
/**
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22950
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98479/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22950
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22612#discussion_r230852432
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -394,9 +394,15 @@ private[spark] object JsonProtocol {
/**
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22612#discussion_r230858827
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -394,9 +394,15 @@ private[spark] object JsonProtocol {
/**
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22950
**[Test build #98479 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98479/testReport)**
for PR 22950 at commit
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/22911
> But whether to store it in secrets is not a question. You either store it
in a secret or you don't support the keytab/principal feature in Spark at all,
and we can delete a bunch of code here.
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22911#discussion_r230860519
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesClusterSchedulerBackend.scala
---
@@ -123,7
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22949#discussion_r230858019
--- Diff: dev/create-release/known_translations ---
@@ -203,3 +203,61 @@ shenh062326 - Shen Hong
aokolnychyi - Anton Okolnychyi
linbojin -
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22949#discussion_r230856955
--- Diff: dev/create-release/known_translations ---
@@ -203,3 +203,61 @@ shenh062326 - Shen Hong
aokolnychyi - Anton Okolnychyi
linbojin -
Github user jianjianjiao commented on the issue:
https://github.com/apache/spark/pull/22926
@mccheah, @smurakozi @vanzin Could you please help take a look at this
PR? Thanks.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22949
**[Test build #98485 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98485/testReport)**
for PR 22949 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22949
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22949
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user lresende commented on the issue:
https://github.com/apache/spark/pull/22867
@gss2002 Sorry I missed this initially, but great that @vanzin is helping
you with the fix.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22949
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98478/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22949
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22949
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22949
**[Test build #98478 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98478/testReport)**
for PR 22949 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #98484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98484/testReport)**
for PR 22612 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22429
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22429
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98475/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22429
**[Test build #98475 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98475/testReport)**
for PR 22429 at commit
Github user kevinyu98 commented on the issue:
https://github.com/apache/spark/pull/22918
@xuanyuanking thanks, I fixed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21688
**[Test build #98483 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98483/testReport)**
for PR 21688 at commit
Github user javierluraschi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22913#discussion_r230833894
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/arrow/ArrowUtils.scala
---
@@ -71,6 +71,7 @@ object ArrowUtils {
case d:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22913
**[Test build #98482 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98482/testReport)**
for PR 22913 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22940
hmhm .. it's trivial and yea it is a logical change. I happened to take a
look some codes around here lately, and the name `SQLUtils` actually annoyed me
few times :(. I will leave it to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22928
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98471/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22948
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22928
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22948
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98474/
Test PASSed.
---
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/22305
Hey @gatorsmile it has been quite a while with no review progress on this.
@BryanCutler has some initial comments but I want to get more people's feedback
before addressing those. Since now 2.4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22948
**[Test build #98474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98474/testReport)**
for PR 22948 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22928
**[Test build #98471 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98471/testReport)**
for PR 22928 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98470/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22590
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98473/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22590
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98470 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98470/testReport)**
for PR 22939 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22590
**[Test build #98473 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98473/testReport)**
for PR 22590 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98469/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98469 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98469/testReport)**
for PR 22944 at commit
Github user shahidki31 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22502#discussion_r230811926
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/HadoopFsRelation.scala
---
@@ -86,10 +89,28 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22288
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/22914
Thank you @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #98481 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98481/testReport)**
for PR 22288 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21919
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21919
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98468/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21919
**[Test build #98468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98468/testReport)**
for PR 21919 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22914
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22612#discussion_r230787617
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -394,9 +394,15 @@ private[spark] object JsonProtocol {
/**
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22914
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22940
I'm neutral on it... seems like a logical change but is there any issue
(like ambiguous imports that are annoying) other than not matching the pattern?
I am not super concerned about back-porting
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22921
**[Test build #98480 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98480/testReport)**
for PR 22921 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22921#discussion_r230784165
--- Diff: R/pkg/R/generics.R ---
@@ -748,7 +748,7 @@ setGeneric("add_months", function(y, x) {
standardGeneric("add_months") })
#' @rdname
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22950
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22950
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22950
**[Test build #98479 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98479/testReport)**
for PR 22950 at commit
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/22950
[MINOR] Fix typos and misspellings
## What changes were proposed in this pull request?
Fix typos and misspellings, per
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22944#discussion_r230781583
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1556,6 +1556,14 @@ class DatasetSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22949
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22949
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/22931
@srowen any update on this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22949
**[Test build #98478 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98478/testReport)**
for PR 22949 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22949
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/22949
[minor] update known_translations
## What changes were proposed in this pull request?
update known_translations after running `translate-contributors.py` during
2.4.0 release
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98477/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22947
**[Test build #98477 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98477/testReport)**
for PR 22947 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22931
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22931
Merged to master/2.4
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22944#discussion_r230772018
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1556,6 +1556,14 @@ class DatasetSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22947
**[Test build #98477 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98477/testReport)**
for PR 22947 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22947
**[Test build #98476 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98476/testReport)**
for PR 22947 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98476/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
201 - 300 of 427 matches
Mail list logo