Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r204885925
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r204885339
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala ---
@@ -470,8 +470,11 @@ class RelationalGroupedDataset
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r204883667
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala ---
@@ -470,8 +470,11 @@ class RelationalGroupedDataset
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21403
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/21403
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93508/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21403
**[Test build #93508 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93508/testReport)**
for PR 21403 at commit
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/21864
appveyor timed out #fml
---
-
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/21474
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93501/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21474
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/21474
**[Test build #93501 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93501/testReport)**
for PR 21474 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21854
**[Test build #93517 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93517/testReport)**
for PR 21854 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21854
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/21854
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/21835
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/21835
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93512/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21835
**[Test build #93512 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93512/testReport)**
for PR 21835 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19773
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93504/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19773
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/19773
**[Test build #93504 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93504/testReport)**
for PR 19773 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21772
**[Test build #93516 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93516/testReport)**
for PR 21772 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21772
retest this please
---
-
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/21752
**[Test build #93515 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93515/testReport)**
for PR 21752 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21752
ok to test
---
-
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/21403
cc @maryannxue Review this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20699
This only handles a corner case. This is not what we expect.
---
-
To unsubscribe, e-mail:
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/21546#discussion_r204868891
--- Diff: python/pyspark/serializers.py ---
@@ -184,27 +184,67 @@ def loads(self, obj):
raise NotImplementedError
-class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21851
**[Test build #93514 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93514/testReport)**
for PR 21851 at commit
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21653
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21851
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/21851
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21440
@squito Thank you!
---
-
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/21851
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/21650
@BryanCutler Thanks for taking a look at this! Yeah I think this works too.
Let me update the code and try it. Thanks again!
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21775
---
-
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/21775
**[Test build #93513 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93513/testReport)**
for PR 21775 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21851
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/21851
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93503/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21851
**[Test build #93503 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93503/testReport)**
for PR 21851 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21775
The commit has been tested.
LGTM Thanks! Merged to master.
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21775
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/21650
I gave it a shot to extract the UDFs in one traversal, using the first
occurrence of either pandas or batch udf. I think it's much clearer
```scala
object ExtractPythonUDFs extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21835
**[Test build #93512 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93512/testReport)**
for PR 21835 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21850#discussion_r204862351
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala
---
@@ -414,6 +414,9 @@ object SimplifyConditionals
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21835
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/21835
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21835#discussion_r204861059
--- Diff: R/pkg/tests/fulltests/test_context.R ---
@@ -21,10 +21,11 @@ test_that("Check masked functions", {
# Check that we are not masking any
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21865
**[Test build #93511 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93511/testReport)**
for PR 21865 at commit
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/21865
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/21865
add to whitelist
---
-
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/21864
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/21864
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93509/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21864
**[Test build #93509 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93509/testReport)**
for PR 21864 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21865
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/21865
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/21865
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user ericfchang opened a pull request:
https://github.com/apache/spark/pull/21865
[SPARK-24895] Remove spotbugs plugin
## What changes were proposed in this pull request?
Spotbugs maven plugin was a recently added plugin before 2.4.0 snapshot
artifacts were broken.
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21850#discussion_r204851642
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyConditionalSuite.scala
---
@@ -61,7 +61,17 @@ class
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21118
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21863
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93506/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21863
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/21863
**[Test build #93506 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93506/testReport)**
for PR 21863 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21118
Thanks! Merged to master.
---
-
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/21857
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/21857
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/21857
**[Test build #93510 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93510/testReport)**
for PR 21857 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21864
**[Test build #93509 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93509/testReport)**
for PR 21864 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21864
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/21864
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
GitHub user shaneknapp opened a pull request:
https://github.com/apache/spark/pull/21864
[SPARK-24908][R] removing spaces to make lintr happy
## What changes were proposed in this pull request?
during my travails in porting spark builds to run on our centos worker, i
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21811
lgtm
will wait a bit for any more comments before merging
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21118
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93500/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21118
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/21403
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/21403
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/21118
**[Test build #93500 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93500/testReport)**
for PR 21118 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21403
**[Test build #93508 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93508/testReport)**
for PR 21403 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21439
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/21439
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93499/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21439
**[Test build #93499 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93499/testReport)**
for PR 21439 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21863
Could you add a test case?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21862
We're holding on all additional pod modification feature requests in favor
of work on https://issues.apache.org/jira/browse/SPARK-24434, which would allow
the pods to be customized arbitrarily.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21608
**[Test build #93507 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93507/testReport)**
for PR 21608 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21861
Thanks for your PR, but we plan to propose new changes in DataSourceV2 APIs
very shortly.
---
-
To unsubscribe, e-mail:
Github user Achuth17 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21608#discussion_r204823567
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/CommandUtils.scala
---
@@ -47,15 +48,26 @@ object CommandUtils extends Logging {
Github user Achuth17 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21608#discussion_r204823533
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/CommandUtils.scala
---
@@ -47,15 +48,26 @@ object CommandUtils extends Logging {
Github user Achuth17 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21608#discussion_r204823422
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -148,6 +148,19 @@ class StatisticsSuite extends
Github user Achuth17 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21608#discussion_r204823485
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningAwareFileIndex.scala
---
@@ -59,14 +59,15 @@ abstract class
Github user Achuth17 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21608#discussion_r204823137
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -148,6 +149,25 @@ class StatisticsSuite extends
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21803
@MaxGekk Thanks for fixing the issue in `SHOW CREATE TABLE`! Could you open
a JIRA and add the JIRA number to this PR. Also, could you add the failed case
in ShowCreateTableSuite?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21863
**[Test build #93506 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93506/testReport)**
for PR 21863 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21863
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/21863
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/21857
**[Test build #93505 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93505/testReport)**
for PR 21857 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21857
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/21863
[SPARK-18874][SQL][FOLLOW-UP] Improvement type mismatched message
## What changes were proposed in this pull request?
How to reproduce:
```sql
create table test1(c1 double, c2 string,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21857
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21851
update the PR description?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21811
SGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93495/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21857
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
201 - 300 of 551 matches
Mail list logo