Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22666
retest this please
---
-
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/22853
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98111/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22853
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/22853
**[Test build #98111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98111/testReport)**
for PR 22853 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22843
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/22843
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98114/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22843
**[Test build #98114 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98114/testReport)**
for PR 22843 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22844
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/22844
**[Test build #98117 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98117/testReport)**
for PR 22844 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22844
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98117/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22844
**[Test build #98117 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98117/testReport)**
for PR 22844 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22843
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/22843
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98113/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22843
**[Test build #98113 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98113/testReport)**
for PR 22843 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22845
**[Test build #98116 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98116/testReport)**
for PR 22845 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22845
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/22845
**[Test build #98115 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98115/testReport)**
for PR 22845 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22845
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98115/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22845
**[Test build #98115 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98115/testReport)**
for PR 22845 at commit
Github user sandeep-katta commented on the issue:
https://github.com/apache/spark/pull/22824
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sandeep-katta commented on the issue:
https://github.com/apache/spark/pull/22824
> I mean its easy to miss if a new "case" is added and "update" mode is not
supported. Even now how about LeftSemi, LeftAnti, FullOuter etc?
Currently Stream stream join is supported
Github user sandeep-katta commented on the issue:
https://github.com/apache/spark/pull/22466
cc @gatorsmile if everything is okay ,this PR can be merged ?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22843
**[Test build #98114 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98114/testReport)**
for PR 22843 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22843
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/22843
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/22843#discussion_r228701261
--- Diff: docs/sparkr.md ---
@@ -717,4 +717,5 @@ You can inspect the search path in R with
[`search()`](https://stat.ethz.ch/R-ma
##
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22843
**[Test build #98113 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98113/testReport)**
for PR 22843 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22843
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/22843
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/22843#discussion_r228701032
--- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
@@ -3477,39 +3447,6 @@ test_that("Window functions on a DataFrame", {
expect_equal(result,
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/22843#discussion_r228701116
--- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
@@ -3477,39 +3447,6 @@ test_that("Window functions on a DataFrame", {
expect_equal(result,
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21710
Sure
From: Huaxin Gao
Sent: Friday, October 26, 2018 1:16:38 PM
To: apache/spark
Cc: Felix Cheung; Mention
Subject: Re:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22608
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98109/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22608
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/22608
**[Test build #98109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98109/testReport)**
for PR 22608 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22666
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98112/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22666
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/22666
**[Test build #98112 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98112/testReport)**
for PR 22666 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22843
Some deprecated functions (e.g., createExternalTable, registerTempTable,
dropTempTable) are mentioned in `Migration Guide` of Spark R document, but
seems some are not. Should we explicitly declare
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22830
cc @mengxr any idea why we have this `formatVersion`?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22666
**[Test build #98112 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98112/testReport)**
for PR 22666 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22853
**[Test build #98111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98111/testReport)**
for PR 22853 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22853
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/22853
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 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):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98107/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22288
**[Test build #98107 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98107/testReport)**
for PR 22288 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19601
Sure, let me close this
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk closed the pull request at:
https://github.com/apache/spark/pull/19601
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20503
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98110/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20503
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/20503
**[Test build #98110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98110/testReport)**
for PR 20503 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20503
**[Test build #98110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98110/testReport)**
for PR 20503 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22624
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98105/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22624
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/22624
**[Test build #98105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98105/testReport)**
for PR 22624 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20503
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22775
Oh you mean the conflict fixing is not that hard. Thanks for doing this
@cloud-fan. I planned to do this today .. :-).
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22608
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/22608
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/22608
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4551/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22608
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4551/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22760
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98108/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22856
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/22760
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/22760
**[Test build #98108 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98108/testReport)**
for PR 22760 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22856
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98106/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22856
**[Test build #98106 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98106/testReport)**
for PR 22856 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22760
Test FAILed.
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/22760
Kubernetes integration test status failure
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4550/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22760
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/22608
**[Test build #98109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98109/testReport)**
for PR 22608 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22760
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4550/
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21157
Yea, so to avoid to break, we could change the default pickler to
CloudPickler or document this workaround. @superbobry, can you check if the
case can be preserved if we use CloudPickler
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22775
Yea, but I meant a bit complicated but I'm okay in that way @cloud-fan.
Thanks for doing that. I planed to do it today (now) :).
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22760
**[Test build #98108 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98108/testReport)**
for PR 22760 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21588
> Hive 2.3 works with Hadoop 2.x (Hive 3.x works with Hadoop 3.x).
This is essentially what we need for Hadoop 3 support
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/22760
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22820
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/22820
merging
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dhunziker commented on the issue:
https://github.com/apache/spark/pull/16812
> This can be easily worked around, no?
It can be worked around, losing the boolean type by converting to string in
the process though. With any workaround it'll be better to get
Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r228681974
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/HadoopDelegationTokenManager.scala
---
@@ -66,7 +66,8 @@ private[spark] class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22854
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98103/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22854
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/22854
**[Test build #98103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98103/testReport)**
for PR 22854 at commit
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22288#discussion_r228678209
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -415,9 +420,54 @@ private[spark] class TaskSchedulerImpl(
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/22608
> Just noticed this, but could you open a separate bug for adding these
tests, instead of re-using the one where the main code was added? It's a large
enough thing that it should be a separate
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/22288#discussion_r228675680
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -415,9 +420,54 @@ private[spark] class TaskSchedulerImpl(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22815
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22608
Just noticed this, but could you open a separate bug for adding these
tests, instead of re-using the one where the main code was added? It's a large
enough thing that it should be a separate thing.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22848
---
-
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/22815
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r228674142
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/HadoopDelegationTokenManager.scala
---
@@ -66,7 +66,8 @@ private[spark] class
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22848
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r228671856
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2328,7 +2328,14 @@ private[spark] object Utils extends Logging {
* configure
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r228669828
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -796,16 +806,57 @@ private[history] class
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r228673330
--- Diff:
core/src/test/scala/org/apache/spark/deploy/history/FsHistoryProviderSuite.scala
---
@@ -413,6 +417,68 @@ class FsHistoryProviderSuite extends
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r228673299
--- Diff:
core/src/test/scala/org/apache/spark/deploy/history/FsHistoryProviderSuite.scala
---
@@ -413,6 +417,68 @@ class FsHistoryProviderSuite extends
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r228672761
--- Diff:
core/src/main/scala/org/apache/spark/util/logging/DriverLogger.scala ---
@@ -0,0 +1,206 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r228671145
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -796,16 +806,57 @@ private[history] class
1 - 100 of 485 matches
Mail list logo