Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20203
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20203
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86490/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20203
**[Test build #86490 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86490/testReport)**
for PR 20203 at commit
[`882bb9e`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20354
**[Test build #86498 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86498/testReport)**
for PR 20354 at commit
[`46f354b`](https://github.com/apache/spark/commit/46
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20354
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20354
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/113/
Test
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/20354
@holdenk @MLnick , just found some minor doc differences between python and
scala during the audit for 2.3
---
-
To unsubscr
GitHub user BryanCutler opened a pull request:
https://github.com/apache/spark/pull/20354
[SPARK-23163][DOC][PYTHON] Sync ML Python API with Scala
## What changes were proposed in this pull request?
This syncs the ML Python API with Scala for differences found after the 2.3
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20203
@attilapiros can you please update the PR description to also mention node
blacklisting (at least briefly) and file a jira for the followup work, and ping
me & tom on it?
lgtm
---
---
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20203
I tried this out on a cluster and seemed fine. I also tried with a bad
app, where all tasks fail, its not just because of the host, and all the
executors show up as blacklisted, but I guess that is
Github user superbobry closed the pull request at:
https://github.com/apache/spark/pull/19992
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/20295#discussion_r163081467
--- Diff: python/pyspark/serializers.py ---
@@ -267,13 +267,13 @@ def load_stream(self, stream):
"""
Deserialize ArrowRecordBat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20352
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86494/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20352
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20352
**[Test build #86494 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86494/testReport)**
for PR 20352 at commit
[`f8ae698`](https://github.com/apache/spark/commit/f
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19671#discussion_r163082224
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1322,33 +1322,55 @@ class BlockManagerSuite extends SparkFunSuite with
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19671#discussion_r163081760
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1322,33 +1322,55 @@ class BlockManagerSuite extends SparkFunSuite with
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/20091
Thanks @jiangxb1987 for the great work !
I will merge this in when the build successfully completes.
---
-
To unsubscribe, e-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20353
**[Test build #86497 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86497/testReport)**
for PR 20353 at commit
[`390cf8b`](https://github.com/apache/spark/commit/39
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20353
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20353
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/112/
Test
GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/20353
[SPARK-20664][core] Follow-up: remove workaround for SPARK-23103.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vanzin/spark SPARK-20664
Alte
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20201
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20201
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86489/
Test PASSed.
---
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163074869
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20201
**[Test build #86489 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86489/testReport)**
for PR 20201 at commit
[`48b9fda`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20352
**[Test build #86494 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86494/testReport)**
for PR 20352 at commit
[`f8ae698`](https://github.com/apache/spark/commit/f8
Github user neilalex commented on the issue:
https://github.com/apache/spark/pull/20352
sure
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apac
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20091
**[Test build #86496 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86496/testReport)**
for PR 20091 at commit
[`ccdc0e6`](https://github.com/apache/spark/commit/cc
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20350
**[Test build #86495 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86495/testReport)**
for PR 20350 at commit
[`fcd665e`](https://github.com/apache/spark/commit/fc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20091
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20091
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/111/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20350
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/110/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20350
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/20352
@neilalex Can you add the code snippet in the PR description as a new test
case ? That way we will ensure this behavior is tested going forward
---
---
Github user MrBago commented on the issue:
https://github.com/apache/spark/pull/20285
I'd like to try and get this patched into 2.3 to make sure our
documentation is complete for the 2.3 release. @viirya and @WeichenXu123 would
you mind having another look at it?
---
--
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20350
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20350
cc @gatorsmile @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/20352
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20352
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20351
**[Test build #86493 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86493/testReport)**
for PR 20351 at commit
[`c75190b`](https://github.com/apache/spark/commit/c7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20352
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user neilalex opened a pull request:
https://github.com/apache/spark/pull/20352
[SPARK-21727][R] Allow multi-element atomic vector as column type in SparkR
DataFrame
## What changes were proposed in this pull request?
A fix to https://issues.apache.org/jira/browse/SP
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20351
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20350
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20350
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86488/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20350
**[Test build #86488 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86488/testReport)**
for PR 20350 at commit
[`fcd665e`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20351
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86491/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20351
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20351
**[Test build #86491 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86491/testReport)**
for PR 20351 at commit
[`c75190b`](https://github.com/apache/spark/commit/c
Github user srowen closed the pull request at:
https://github.com/apache/spark/pull/20301
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user squito commented on the issue:
https://github.com/apache/spark/pull/19992
thanks for looking into this @superbobry -- can you actually close this
yourself? we can't directly close it (there is a way but its more complicated)
---
-
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163047551
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -123,6 +123,16 @@ package object config {
.timeConf(TimeUnit.MIL
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163051028
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163053874
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/CacheRecoveryIntegrationSuite.scala
---
@@ -0,0 +1,136 @@
+/*
+ * Licensed to the Apache So
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163048247
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163052079
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163042811
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163055034
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/CacheRecoveryIntegrationSuite.scala
---
@@ -0,0 +1,136 @@
+/*
+ * Licensed to the Apache So
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163050134
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163052565
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163045943
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163045008
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163052910
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163041732
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163054043
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/CacheRecoveryIntegrationSuite.scala
---
@@ -0,0 +1,136 @@
+/*
+ * Licensed to the Apache So
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163053606
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/CacheRecoveryIntegrationSuite.scala
---
@@ -0,0 +1,136 @@
+/*
+ * Licensed to the Apache So
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163050661
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163042297
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163046520
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163045368
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163046590
--- Diff: core/src/main/scala/org/apache/spark/CacheRecoveryManager.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licensed to the Apache Software Foundation (A
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163052829
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163052459
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19041#discussion_r163050512
--- Diff:
core/src/test/scala/org/apache/spark/CacheRecoveryManagerSuite.scala ---
@@ -0,0 +1,200 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19872
**[Test build #86492 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86492/testReport)**
for PR 19872 at commit
[`cc659bc`](https://github.com/apache/spark/commit/cc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19872
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/109/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19872
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20344
---
-
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/20344
Merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20344
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20344
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86486/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20344
**[Test build #86486 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86486/testReport)**
for PR 20344 at commit
[`78c5ef5`](https://github.com/apache/spark/commit/7
Github user skambha commented on the issue:
https://github.com/apache/spark/pull/17185
sure. Let me look into it. Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19872
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86487/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19872
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19872
**[Test build #86487 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86487/testReport)**
for PR 19872 at commit
[`91885e5`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20351
**[Test build #86491 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86491/testReport)**
for PR 20351 at commit
[`c75190b`](https://github.com/apache/spark/commit/c7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20351
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user jose-torres opened a pull request:
https://github.com/apache/spark/pull/20351
[SPARK-23014][SS] Fully remove V1 memory sink.
## What changes were proposed in this pull request?
Fully remove the V1 memory sink, now that all kinds of execution support
the V2 versi
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/19993
Well yes it would - but the method checks inputCols/inputCol first so will
always fail for that reason here, ie we arenât actually testing the full
code path
On Mon, 22 Jan 2018 at 16:43,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20344
**[Test build #4071 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4071/testReport)**
for PR 20344 at commit
[`78c5ef5`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20203
**[Test build #86490 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86490/testReport)**
for PR 20203 at commit
[`882bb9e`](https://github.com/apache/spark/commit/88
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/20203
Jenkins, test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20343#discussion_r163030125
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/TPCDSQuerySuite.scala ---
@@ -244,7 +244,7 @@ class TPCDSQuerySuite extends BenchmarkQueryTest
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20330
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20342
@maropu That is blocked by the design and impl of usage log.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20330
Merging to master / 2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20343
Regarding the change you made in `[date] + 14 days -> date + INTERVAL 14
days`, could we first support it before we merge this PR? It sounds like this
is trivial to support, right?
---
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20343
Regarding the updates of `TPCDSQueryBenchmark`, we can do it in a separate
PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr
201 - 300 of 528 matches
Mail list logo