Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21504
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91570/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21504
**[Test build #91570 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91570/testReport)**
for PR 21504 at commit
[`421e16b`](https://github.com/apache/spark/commit/4
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/21277
Closing the PR due to the discussions in the JIRA,
https://issues.apache.org/jira/browse/SPARK-15784 and the PR
https://github.com/apache/spark/pull/21493
---
-
Github user shahidki31 closed the pull request at:
https://github.com/apache/spark/pull/21277
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20697
**[Test build #91576 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91576/testReport)**
for PR 20697 at commit
[`f67d582`](https://github.com/apache/spark/commit/f6
Github user ssuchter commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r194150705
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/KubernetesSuite.scala
---
@@ -0,0 +1,2
Github user ssuchter commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r194150614
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/Utils.scala
---
@@ -0,0 +1,88 @@
+
Github user ssuchter commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r194150624
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/Utils.scala
---
@@ -0,0 +1,88 @@
+
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21508
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91569/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21508
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/21508
**[Test build #91569 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91569/testReport)**
for PR 21508 at commit
[`f174263`](https://github.com/apache/spark/commit/f
Github user pgandhi999 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21468#discussion_r194148280
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
---
@@ -813,8 +813,14 @@ private[spark] class Client(
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21247#discussion_r194147472
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -138,3 +121,40 @@ private[sql] class JSONOptions(
Github user yuchaoran2011 commented on the issue:
https://github.com/apache/spark/pull/21398
I'm not confused about anything.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21398
You said: "even when LOCATION is not specified, Sentry would still not
allow the table to be created."
I said that if that's the case, then your user doesn't have the needed
permission. Beca
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19498
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user TomaszGaweda commented on the issue:
https://github.com/apache/spark/pull/21360
I've tesed it with my application that had problem with predicate pushdowns
to database. Looks good, performance is degradated a bit, but it was previously
ran on Spark 2.3, not 2.4. However, m
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19498
Thanks @rxin, @tdas, @holdenk and @felixcheung.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19498
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19498
Thanks for reviewing this @rxin. Will merge this only to master given the
concerns here.
---
-
To unsubscribe, e-mail: revie
Github user shahidki31 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21509#discussion_r194144115
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/PowerIterationClustering.scala
---
@@ -166,6 +166,7 @@ class PowerIterationClustering priv
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21508
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user yuchaoran2011 commented on the issue:
https://github.com/apache/spark/pull/21398
You said "If that's the case then your user does not have permission to
create the table." What I'm saying is the user did have the permission.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21508
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91568/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21508
**[Test build #91568 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91568/testReport)**
for PR 21508 at commit
[`46c4a55`](https://github.com/apache/spark/commit/4
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/21465#discussion_r194143768
--- Diff: python/pyspark/ml/classification.py ---
@@ -1251,26 +1256,33 @@ class GBTClassifier(JavaEstimator, HasFeaturesCol,
HasLabelCol, HasPredictionCol
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21092
---
-
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/21398
Yes, that's exactly what I said. I'm not sure what part is confusing you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.
Github user yuchaoran2011 commented on the issue:
https://github.com/apache/spark/pull/21398
I can double check. But I remember clearly that the user did have all the
necessary privileges. Without this patch, a permission error was thrown. With
this patch it worked fine. The CREATE TA
Github user zecevicp commented on the issue:
https://github.com/apache/spark/pull/21109
There's no design doc. I didn't feel the change was big enough to warrant
one.
1. Currently there is no spill-over to disk. If the range is too big, users
can switch this off and use the m
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21082
Yea, let's leave this open for few more days in case someone has more
comments.
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21467
Seems fine. will take a look before merginf this in. Thanks for working on
this.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21482
**[Test build #91575 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91575/testReport)**
for PR 21482 at commit
[`559900a`](https://github.com/apache/spark/commit/55
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21082
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/3858/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21082
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21398
If that's the case then your user does not have permission to create the
table. The issue that my patch works around is a bug in Sentry where creating a
table *with the location specified to the same
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/21082
Thanks @HyukjinKwon for your review! @ueshin Do you want to take another
look too?
---
-
To unsubscribe, e-mail: reviews-unsu
Github user yuchaoran2011 commented on the issue:
https://github.com/apache/spark/pull/21398
@vanzin But in Spark SQL, even when LOCATION is not specified, Sentry would
still not allow the table to be created.
---
-
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21082#discussion_r194133638
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -424,6 +424,21 @@ abstract class SparkStrategies extends
Q
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21082#discussion_r194133573
--- Diff: python/pyspark/worker.py ---
@@ -128,6 +128,17 @@ def wrapped(*series):
return lambda *a: (wrapped(*a), arrow_return_type)
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21398
I'm kinda neutral on what should happen, except maybe for filing a Sentry
bug. I'll let @cloud-fan decide whether this makes sense in Spark.
For the Sentry bug, there's an easy way to reprodu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21082
**[Test build #91574 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91574/testReport)**
for PR 21082 at commit
[`6350408`](https://github.com/apache/spark/commit/63
Github user ssuchter commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r194133234
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/KubernetesSuite.scala
---
@@ -0,0 +1,2
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21511
> But I am concerned about it setting a precedent for having to change
Spark's source code every time you want to request a new resource type. I am
very familiar with the space of machine learning a
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/21283
Thanks @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21082#discussion_r194130366
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/PythonUDF.scala
---
@@ -34,7 +34,12 @@ object PythonUDF {
e.isI
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/21248
Thanks @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user huaxingao commented on the issue:
https://github.com/apache/spark/pull/21119
@mengxr @WeichenXu123 I will close this one and submit a new PR soon.
Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr..
Github user huaxingao closed the pull request at:
https://github.com/apache/spark/pull/21119
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/19498
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.o
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21082#discussion_r194129021
--- Diff: python/pyspark/sql/tests.py ---
@@ -5181,6 +5190,235 @@ def test_invalid_args(self):
'mixture.*aggregate function.*grou
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r194128800
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/Utils.scala
---
@@ -0,0 +1,88 @@
+/
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r194128687
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/Utils.scala
---
@@ -0,0 +1,88 @@
+/
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/21511#discussion_r194128417
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/BasicExecutorFeatureStep.scala
---
@@ -172,7 +184,7 @@ p
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21427
yea, I generally agree with that and I feel in the same way. I think I had
a talk about this @gatorsmile and @cloud-fan multiple times. Here is my
thought: we should not make a configuration onl
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/21119
@huaxingao Create a new PR is better I think.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additio
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/21427
@HyukjinKwon I agree with you 99% people will mostly certainly not use the
config. I think @gatorsmile 's concern is that in the rare case that some
people are actually depending on the existing
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21481
Since it is Java bytecode analysis, it is available for Scala code, too.
In my quick test, findBugs overlooked a possible overflow. On the other
hand, findBugs found another redundant null check.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21092
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3712/
---
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/20697#discussion_r194124004
--- Diff:
resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/KubernetesSuite.scala
---
@@ -0,0 +1
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21247#discussion_r194123992
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -138,3 +121,40 @@ private[sql] class JSONOptions(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21092
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3712/
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21092
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/21092
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/3857/
Tes
Github user huaxingao commented on the issue:
https://github.com/apache/spark/pull/21119
@mengxr Sorry for the delay. I will submit an update later today. Do you
want me to close this PR and do a new one? or just update this PR?
---
-
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21247#discussion_r194117346
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -138,3 +121,40 @@ private[sql] class JSONOptions(
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21247#discussion_r194119938
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -2427,4 +2427,14 @@ class JsonSuite extends QueryT
Github user NihalHarish commented on a diff in the pull request:
https://github.com/apache/spark/pull/21482#discussion_r194119670
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NullExpressionsSuite.scala
---
@@ -56,6 +56,16 @@ class NullExpressions
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21483
Thank you @felixcheung, @mengxr and @BryanCutler for reviewing this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21427
I'm okay if that's only the way to get through here. but I must say I
wonder who's going to intentionally switch this off though. This now sounds
more like a bug or a design issue to be fixed to
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21481
is findBugs available for scala code as well?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21109
do we have a design doc? I have a couple of high-level questions:
1. what if the range is big and the queue OOM?
2. can't we apply it with cartesian join if there is no equal condition?
and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21092
**[Test build #91573 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91573/testReport)**
for PR 21092 at commit
[`a61d897`](https://github.com/apache/spark/commit/a6
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r194117385
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/KubernetesConfSuite.scala
---
@@ -88,15 +90,42 @@ class Kubernete
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21468
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/21045
**[Test build #91572 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91572/testReport)**
for PR 21045 at commit
[`8d3a838`](https://github.com/apache/spark/commit/8d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21468
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91571/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21468
**[Test build #91571 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91571/testReport)**
for PR 21468 at commit
[`5e733ae`](https://github.com/apache/spark/commit/5
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21483
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r194109752
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/KubernetesConfSuite.scala
---
@@ -87,11 +89,37 @@ class KubernetesCo
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r194112328
--- Diff: docs/running-on-kubernetes.md ---
@@ -624,4 +624,20 @@ specific to Spark on Kubernetes.
spark.kubernetes.executor.secrets.ENV_VAR=spark-sec
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r194113403
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/KubernetesConfSuite.scala
---
@@ -88,15 +90,42 @@ class KubernetesCo
Github user mengxr commented on the issue:
https://github.com/apache/spark/pull/21119
@huaxingao Any updates?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user mengxr commented on the issue:
https://github.com/apache/spark/pull/21483
LGTM. Merged into master. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21468
**[Test build #91571 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91571/testReport)**
for PR 21468 at commit
[`5e733ae`](https://github.com/apache/spark/commit/5e
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21045#discussion_r194109519
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -128,6 +128,173 @@ case class MapKeys(ch
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r194107584
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala
---
@@ -154,6 +176,24 @@ private[spark] object Config e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21045
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/21045
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91565/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21045
**[Test build #91565 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91565/testReport)**
for PR 21045 at commit
[`bbc20ee`](https://github.com/apache/spark/commit/b
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/21501#discussion_r194098947
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StopWordsRemover.scala ---
@@ -84,7 +86,28 @@ class StopWordsRemover @Since("1.5.0") (@Since("1.5.
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/21501#discussion_r194099298
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StopWordsRemover.scala ---
@@ -84,7 +86,28 @@ class StopWordsRemover @Since("1.5.0") (@Since("1.5.
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21502#discussion_r194101781
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala
---
@@ -153,6 +154,23 @@ class BroadcastJoinSuite extends
Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21504#discussion_r194101270
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/StreamingQueryManager.scala
---
@@ -55,6 +57,19 @@ class StreamingQueryManager priva
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21504
**[Test build #91570 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91570/testReport)**
for PR 21504 at commit
[`421e16b`](https://github.com/apache/spark/commit/42
Github user tomhanetz commented on the issue:
https://github.com/apache/spark/pull/21512
@abellina No, the page stays at the same view as it was before the refresh.
It is the same functionality as hitting the browser refresh button manually.
---
-
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/21427
I ran into @ueshin and @gatorsmile at the Summit. It seems the preferable
way to move forward is to having a configuration to fall back to the existing
behavior and change the default behavior to
Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21504#discussion_r194100709
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/StreamingQueryManager.scala
---
@@ -55,6 +57,19 @@ class StreamingQueryManager priva
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21508
**[Test build #91569 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91569/testReport)**
for PR 21508 at commit
[`f174263`](https://github.com/apache/spark/commit/f1
Github user bkrieger commented on a diff in the pull request:
https://github.com/apache/spark/pull/21508#discussion_r194082850
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1568,11 +1568,13 @@ class Analyzer(
expr.fi
501 - 600 of 713 matches
Mail list logo