Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22090#discussion_r210772634
--- Diff: docs/mllib-evaluation-metrics.md ---
@@ -461,11 +461,11 @@ $$rel_D(r) = \begin{cases}1 & \text{if $r \in D$}, \\
0 & \text{otherwise}.\end{
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22090#discussion_r210772686
--- Diff: docs/mllib-evaluation-metrics.md ---
@@ -461,11 +461,11 @@ $$rel_D(r) = \begin{cases}1 & \text{if $r \in D$}, \\
0 & \text{otherwise}.\end{
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22125
**[Test build #4279 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4279/testReport)**
for PR 22125 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21584
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/21584
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94862/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21584
**[Test build #94862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94862/testReport)**
for PR 21584 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22125
OK, we should bundle these, but w/e
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22125
**[Test build #4279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4279/testReport)**
for PR 22125 at commit
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/20725
still having problems getting this to pass:
```
[error] (sql-kafka-0-10/test:test) sbt.TestsFailedException: Tests
unsuccessful
[error] (core/test:test) sbt.TestsFailedException:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22126
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94863/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22126
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/22126
**[Test build #94863 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94863/testReport)**
for PR 22126 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22128
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22128
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user bomeng commented on the issue:
https://github.com/apache/spark/pull/22127
Good points. I will leave it open for any suggestions for improving the
user experience..
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22128
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user cclauss opened a pull request:
https://github.com/apache/spark/pull/22128
Add test_slice() to streaming BasicOperations
As suggested in
https://github.com/apache/spark/pull/20838#pullrequestreview-139118618
## What changes were proposed in this pull request?
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21909#discussion_r210767018
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -2223,21 +2223,31 @@ class JsonSuite extends
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22108
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21909#discussion_r210765672
--- Diff: docs/sql-programming-guide.md ---
@@ -1894,6 +1894,7 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
- In
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22108
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22127
@bomeng There seems to be bit of history to this :-) . Please check
https://github.com/apache/spark/pull/15011
where we decided against silently switching to "default" database.
---
Github user steveloughran closed the pull request at:
https://github.com/apache/spark/pull/22117
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/22081
Thanks. Two less JARs on the CP to keep up to date âwhat more can anyone
want?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21950
**[Test build #94869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94869/testReport)**
for PR 21950 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21135
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21561
---
-
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/21561
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22112
Thanks for the clarification, but I guess my point is with your last
statement:
> - with assumption that we will expand solution to cover all later.
If we document this and say
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21909
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94860/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21909
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/21909
**[Test build #94860 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94860/testReport)**
for PR 21909 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22127
**[Test build #94868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94868/testReport)**
for PR 22127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22127
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/22127
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
GitHub user bomeng opened a pull request:
https://github.com/apache/spark/pull/22127
[SPARK-25032][SQL] fix drop database issue
## What changes were proposed in this pull request?
When user tries to drop the current database (other than default database),
after the database is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22114
**[Test build #94867 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94867/testReport)**
for PR 22114 at commit
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/22112
@tgravescs I was specifically in agreement with
> Personally I don't want to talk about implementation until we decide what
we want our semantics to be around the unordered operations because
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22114
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user NiharS commented on the issue:
https://github.com/apache/spark/pull/22114
Tried with a significantly larger input, both with and without the change.
They ran in just about the same time.
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21990
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21990
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94866/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21990
**[Test build #94866 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94866/testReport)**
for PR 21990 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21990
**[Test build #94866 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94866/testReport)**
for PR 21990 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21950
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94853/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21950
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21950
**[Test build #94853 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94853/testReport)**
for PR 21950 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22124
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22124
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94861/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22124
**[Test build #94861 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94861/testReport)**
for PR 22124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21221
**[Test build #94865 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94865/testReport)**
for PR 21221 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22107
@felixcheung I have incorporated the comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/22126#discussion_r210724650
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/HigherOrderFunctionsSuite.scala
---
@@ -363,9 +363,9 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22045
**[Test build #94864 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94864/testReport)**
for PR 22045 at commit
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22112
@mridulm so just to clarify are you agreeing that we need to decide on
what we do with zip and others or are you agreeing that we should document
these as unordered actions thus retries might be
Github user ajacques closed the pull request at:
https://github.com/apache/spark/pull/21889
---
-
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/21584
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2256/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22126
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/21584
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/22126
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22126
LGTM
---
-
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/21584
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22081
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user ajacques commented on the issue:
https://github.com/apache/spark/pull/21889
Thanks for the response all. @mailman If it's really your preference, I
will create a PR against that branch and close this one. My intention was never
to take away from your efforts, and I still
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21584
Let's merge at the end of the day pacific time (~5PM-ish) on Friday, August
17, pending any additional feedback on the mailing list thread discussing the
subject of including this in 2.4.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22081
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21584
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2256/
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22119
+1 for @tgravescs 's comments. In terms of usability, the mapping and
configuration will be easier for the most customers.
For the following @gengliangwang 's comment, technically
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/22112
I agree @tgravescs, I was looking at the implementation to understand what
the expectations are wrt newly introduced methods/fields and whether they make
sense : I did not see any details
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22123
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94854/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22123
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/22123
**[Test build #94854 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94854/testReport)**
for PR 22123 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22126
**[Test build #94863 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94863/testReport)**
for PR 22126 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22126
cc @mn-mikke
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/22126
[SPARK-23938][SQL][FOLLOW-UP][TEST] Nullabilities of value arguments should
be true.
## What changes were proposed in this pull request?
This is a follow-up pr of #22017 which added
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21584
**[Test build #94862 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94862/testReport)**
for PR 21584 at commit
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/21584
This PR has been updated to pass Jenkins by removing the `with RTestsSuite`
line in `KubernetesSuite`. As such, this feature may be merged and the `with
RTestsSuite` will be re-included in a
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/21320
> I see no point of leaving this PR open.
I don't agree with you on that point, and I've expressed my view in
https://github.com/apache/spark/pull/21889#issuecomment-413655304.
---
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/21889
Essentially, this PR was created to take the management of #21320 out of my
hands, with a view towards facilitating its incorporation into Spark 2.4. It
was my suggestion, one based in frustration.
Github user KraFusion commented on the issue:
https://github.com/apache/spark/pull/22125
@kiszk PR created yesterday for ```configurations.md```
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22125
Thanks, would it possible to address similar issues? For example, in
`configurations.md`.
---
-
To unsubscribe, e-mail:
Github user vackosar commented on a diff in the pull request:
https://github.com/apache/spark/pull/21919#discussion_r210708107
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/progress.scala ---
@@ -254,3 +259,10 @@ class SinkProgress protected[sql](
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22125
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22125
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22125
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user arunmahadevan commented on the issue:
https://github.com/apache/spark/pull/21919
LGTM overall except one minor comment.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21919#discussion_r210707152
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/progress.scala ---
@@ -254,3 +259,10 @@ class SinkProgress protected[sql](
GitHub user KraFusion opened a pull request:
https://github.com/apache/spark/pull/22125
[DOCS] Fix cloud-integration.md Typo
Corrected typo; changed spark-default.conf to spark-defaults.conf
You can merge this pull request into a Git repository by running:
$ git pull
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/22117
Test failure in `
org.apache.spark.sql.hive.client.HiveClientSuites.(It is not a test it is a
sbt.testing.SuiteSelector)`:
```
Caused by: sbt.ForkMain$ForkError:
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21909#discussion_r210704902
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -2223,21 +2223,31 @@ class JsonSuite extends
Github user arunmahadevan commented on the issue:
https://github.com/apache/spark/pull/21819
@HyukjinKwon , can you take it forward? Appreciate your effort and thanks
in advance.
---
-
To unsubscribe, e-mail:
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22124
cc @gengliangwang
---
-
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/22117
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94851/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22117
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/22117
**[Test build #94851 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94851/testReport)**
for PR 22117 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22124
**[Test build #94861 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94861/testReport)**
for PR 22124 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22124
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/22124
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
GitHub user wangyum opened a pull request:
https://github.com/apache/spark/pull/22124
[SPARK-25135][SQL] Insert datasource table may all null when select from
view
## What changes were proposed in this pull request?
How to reproduce:
```scala
val path =
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22081
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
101 - 200 of 440 matches
Mail list logo