Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18029
**[Test build #83144 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83144/testReport)**
for PR 18029 at commit
[`2c2a56a`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18029
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83144/
Test PASSed.
---
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19588#discussion_r147542224
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/VectorIndexer.scala ---
@@ -311,22 +342,39 @@ class VectorIndexerModel private[ml] (
Github user jmchung commented on a diff in the pull request:
https://github.com/apache/spark/pull/19567#discussion_r147542014
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/PostgresIntegrationSuite.scala
---
@@ -134,11 +149,28 @@ class Postgre
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19567
**[Test build #83145 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83145/testReport)**
for PR 19567 at commit
[`edc5f6f`](https://github.com/apache/spark/commit/ed
Github user shaolinliu commented on the issue:
https://github.com/apache/spark/pull/17581
ok.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18029
**[Test build #83144 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83144/testReport)**
for PR 18029 at commit
[`2c2a56a`](https://github.com/apache/spark/commit/2c
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19122
@jkbradley Sure I will!
---
-
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/19451
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/19451
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83135/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19451
**[Test build #83135 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83135/testReport)**
for PR 19451 at commit
[`40d8797`](https://github.com/apache/spark/commit/4
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18687
@gatorsmile Any suggestion for this issue? Leave it as it for now?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19593
**[Test build #83142 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83142/testReport)**
for PR 19593 at commit
[`ea3943e`](https://github.com/apache/spark/commit/ea
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19570#discussion_r147540587
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1753,6 +1753,27 @@ class Dataset[T] private[sql](
*
* Also as sta
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19570
**[Test build #83143 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83143/testReport)**
for PR 19570 at commit
[`977c78e`](https://github.com/apache/spark/commit/97
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/19593
[WIP][SPARK-22374][SQL][2.2] closeAllForUGI is required after using
loginUserFromKeytab
## What changes were proposed in this pull request?
In a secure cluster, FileSystem.Cache grow
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/13206
We still maintain a version of this for our Spark builds to avoid an extra
sort in Hive. If someone is willing to review it, I can probably find the time
to rebase it on master. I think the year this
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19570#discussion_r147540274
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1753,6 +1753,27 @@ class Dataset[T] private[sql](
*
* Also as sta
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19592
**[Test build #83141 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83141/testReport)**
for PR 19592 at commit
[`9744e77`](https://github.com/apache/spark/commit/97
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19592
cc @HyukjinKwon @ueshin @BryanCutler @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19442
Before the discussion of attribute API is finalized, I think the review
won't be started. Close this for now and maybe re-open in the future.
---
---
Github user viirya closed the pull request at:
https://github.com/apache/spark/pull/19442
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya closed the pull request at:
https://github.com/apache/spark/pull/18687
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18687
Sure.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apach
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17973
@mikkokupsu We do not plan to support this after an offline discussion with
the other Committers. Thanks for your contribution. Could you close this PR
please?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19592
**[Test build #83140 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83140/testReport)**
for PR 19592 at commit
[`1c523d1`](https://github.com/apache/spark/commit/1c
Github user wangyum closed the pull request at:
https://github.com/apache/spark/pull/18841
---
-
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/13206
@rdblue What is the latest status of this PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15049
cc @jiangxb1987
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17581
@shaolinliu The new behavior introduced in this PR is not clean in the long
term. We might not want to introduce such a change. You can keep it in your
internal fork. Could you close this PR?
-
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18687
@viirya Could you close this PR?
---
-
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/19592
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/19592
**[Test build #83139 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83139/testReport)**
for PR 19592 at commit
[`0515435`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19592
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83139/
Test FAILed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18841
Different DB systems have different behaviors. We might not accept this
behavior change now, until we support the profiles. Could you close this PR?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19592
**[Test build #83139 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83139/testReport)**
for PR 19592 at commit
[`0515435`](https://github.com/apache/spark/commit/05
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18828
ping @rxin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/19592
[SPARK-22347][SQL][PySpark] Support optionally running PythonUDFs in
conditional expressions
## What changes were proposed in this pull request?
Under the current execution mode of Python U
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17174
ping @tanejagagan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18106
This is close to merge. Could you resolve the conflicts? Then, I will
review it. Thanks!
---
-
To unsubscribe, e-mail: review
Github user umehrot2 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19426#discussion_r147537143
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -406,6 +406,41 @@ private[spark] class Executor(
task.metrics.set
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18812
cc @liufengdb Is this PR in the right track?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18766
cc @liancheng Should we still support this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user huaxingao commented on the issue:
https://github.com/apache/spark/pull/17819
@AFractalThought @viirya
I have made changes for QuantileDiscretizer based on this PR. Once this PR
is merged, I will open a jira to submit the PR for QuantileDiscretizer.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18377
@bdrillard Sorry, we do not plan to backport this PR to the maintenance
branch 2.2. Could you please close this PR? Thanks!
---
-
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17953
@LantaoJin Maybe close it now? You can reopen it when the comment is
resolved?
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19559
**[Test build #83138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83138/testReport)**
for PR 19559 at commit
[`a57fb81`](https://github.com/apache/spark/commit/a5
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13706
@lianhuiwang Could you restart the work? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18581
@HyukjinKwon Could you address the conflicts? Then, I will review it
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18642
Since the fix is not right, @jaceklaskowski Could you close this PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19587
Could you add a comment in `abstract class QueryPlan`'s conf? Hopefully,
the others will read the comment before using it.
---
-
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/18610
A couple questions:
1. do we actually *need* to save the initialModel when we persist the
current model? I'm not sure it's necessary and it adds complexity. Also, we
could add it in the futu
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/18610#discussion_r147532256
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala ---
@@ -226,6 +246,12 @@ class LinearRegression @Since("1.3.0")
(@Since
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19587
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19591
**[Test build #83137 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83137/testReport)**
for PR 19591 at commit
[`928a052`](https://github.com/apache/spark/commit/92
GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/19591
[SPARK-11035][core] Add in-process Spark app launcher.
This change adds a new launcher that allows applications to be run
in a separate thread in the same process as the calling code. To
achi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19567#discussion_r147531601
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/PostgresIntegrationSuite.scala
---
@@ -134,11 +149,28 @@ class Post
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19583
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/19583
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83136/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19583
**[Test build #83136 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83136/testReport)**
for PR 19583 at commit
[`7f538df`](https://github.com/apache/spark/commit/7
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19531#discussion_r147530024
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/JoinEstimation.scala
---
@@ -157,64 +154,100 @@ case c
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19531#discussion_r147529424
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/JoinEstimation.scala
---
@@ -157,64 +154,100 @@ case c
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19531#discussion_r147529228
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/JoinEstimation.scala
---
@@ -28,45 +28,43 @@ import
o
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19122
Whoops, could you please send a follow-up PR to do 1 doc update?
* Update https://github.com/apache/spark/blob/master/docs/ml-tuning.md to
say this is supported in Python. (Search for "NOTE: t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19583
**[Test build #83136 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83136/testReport)**
for PR 19583 at commit
[`7f538df`](https://github.com/apache/spark/commit/7f
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19122
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/19583#discussion_r147522958
--- Diff: core/src/main/scala/org/apache/spark/HeartbeatReceiver.scala ---
@@ -51,7 +51,26 @@ private case class ExecutorRegistered(executorId: String)
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/19583#discussion_r147526776
--- Diff: core/src/main/scala/org/apache/spark/HeartbeatReceiver.scala ---
@@ -51,7 +51,26 @@ private case class ExecutorRegistered(executorId: String)
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19122
LGTM
Merging with master
Thanks @WeichenXu123 , @BryanCutler and @viirya !
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/19583#discussion_r147525074
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -241,6 +243,21 @@ final class ShuffleBlockFetcherIterator(
Github user squito commented on the issue:
https://github.com/apache/spark/pull/19583
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/19583#discussion_r147526524
--- Diff: core/src/test/scala/org/apache/spark/HeartbeatReceiverSuite.scala
---
@@ -225,6 +270,7 @@ class HeartbeatReceiverSuite
Matchers.eq(Arr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19590
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user juanrh opened a pull request:
https://github.com/apache/spark/pull/19590
[WIP][SPARK-22148][CORE] TaskSetManager.abortIfCompletelyBlacklisted should
not abort when all current executors are blacklisted but dynamic allocation is
enabled
## What changes were proposed in t
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18544
The failures are related to the changes you made here. Could you fix them?
Thanks!
---
-
To unsubscribe, e-mail: reviews-unsu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19451
**[Test build #83135 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83135/testReport)**
for PR 19451 at commit
[`40d8797`](https://github.com/apache/spark/commit/40
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19585#discussion_r147520400
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -841,15 +852,7 @@ class Analyzer(
cas
Github user sathiyapk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147518995
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceExceptWithFilter.scala
---
@@ -0,0 +1,104 @@
+/*
+ * License
Github user sathiyapk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147518757
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceExceptWithFilter.scala
---
@@ -0,0 +1,104 @@
+/*
+ * License
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18544
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/19587
LGTM, good catch!
---
-
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/18544
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83134/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18544
**[Test build #83134 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83134/testReport)**
for PR 18544 at commit
[`c41475e`](https://github.com/apache/spark/commit/c
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19480
---
-
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/19480
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19451
Please change the title to `[SPARK-22181][SQL]Adds
ReplaceExceptWithNotFilter rule`
---
-
To unsubscribe, e-mail: reviews-uns
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19451
LGMT except a few minor comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147510054
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceOperatorSuite.scala
---
@@ -50,6 +52,108 @@ class ReplaceOperatorSu
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147508919
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceExceptWithFilter.scala
---
@@ -0,0 +1,104 @@
+/*
+ * Licens
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147508867
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceExceptWithFilter.scala
---
@@ -0,0 +1,104 @@
+/*
+ * Licens
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147507850
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceExceptWithFilter.scala
---
@@ -0,0 +1,104 @@
+/*
+ * Licens
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/19439
@WeichenXu123 @viirya @srowen @MrBago @holdenk @HyukjinKwon @dakirsa
@hhbyyh @drdarshan Thank you for taking a look at the PR merging the
"spark-images" spark package into spark and your great
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147507755
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceExceptWithFilter.scala
---
@@ -0,0 +1,104 @@
+/*
+ * Licens
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r147507461
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceExceptWithFilter.scala
---
@@ -0,0 +1,104 @@
+/*
+ * Licens
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19578
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/19578
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83133/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19578
**[Test build #83133 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83133/testReport)**
for PR 19578 at commit
[`c572154`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19573
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/19573
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83131/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19573
**[Test build #83131 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83131/testReport)**
for PR 19573 at commit
[`c0ecbee`](https://github.com/apache/spark/commit/c
101 - 200 of 388 matches
Mail list logo