Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19722
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/19722
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83724/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19722
**[Test build #83724 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83724/testReport)**
for PR 19722 at commit
[`c359644`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19722
**[Test build #83725 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83725/testReport)**
for PR 19722 at commit
[`4ee95d4`](https://github.com/apache/spark/commit/4e
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19722
Yes, it looked so. Let me give a try in my local and push a commit to check.
---
-
To unsubscribe, e-mail: reviews-unsubscr..
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19722
I think I need to make sure it reduces the time significantly before cc'ing
someone to review. I expect the decreas of 15ish mins.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19722
**[Test build #83724 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83724/testReport)**
for PR 19722 at commit
[`c359644`](https://github.com/apache/spark/commit/c3
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/19722
[WIP][SPARK-21693][R][ML] Reduce max iterations in Linear SVM test in R to
speed up AppVeyor build
## What changes were proposed in this pull request?
This PR proposes to reduce max it
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19713
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/19713
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83721/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19713
**[Test build #83721 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83721/testReport)**
for PR 19713 at commit
[`c491974`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83722/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19702
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/19702
**[Test build #83722 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83722/testReport)**
for PR 19702 at commit
[`ba16a5e`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19690
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83720/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19690
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/19690
**[Test build #83720 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83720/testReport)**
for PR 19690 at commit
[`37fba60`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83723/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
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/19719
**[Test build #83723 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83723/testReport)**
for PR 19719 at commit
[`e37c1a8`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83723 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83723/testReport)**
for PR 19719 at commit
[`e37c1a8`](https://github.com/apache/spark/commit/e3
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19689
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user setjet commented on the issue:
https://github.com/apache/spark/pull/18113
Exactly my point. I'll return -/+ inf then for doubles only, and min/max
values for longs
---
-
To unsubscribe, e-mail: reviews
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19720
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83719/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19720
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19689
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18113
BTW let's not change the existing type sum, it already follows mathematical
standard as we expected.
---
-
To unsubscribe, e-m
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19720
**[Test build #83719 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83719/testReport)**
for PR 19720 at commit
[`1722d12`](https://github.com/apache/spark/commit/1
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18113
doubles has -inf and +inf, can we use that?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
Github user setjet commented on the issue:
https://github.com/apache/spark/pull/18113
Ok sounds good. What about doubles? We could return the proper mathematical
defintion, but that is not consistent with Longs
---
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18113
sounds good. I think the deal is, typed sum/count/max/min should follow
mathematical standard instead of sql.
---
-
To unsubsc
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19077#discussion_r150381892
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -48,6 +49,15 @@ public long size() {
}
/*
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19702
**[Test build #83722 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83722/testReport)**
for PR 19702 at commit
[`ba16a5e`](https://github.com/apache/spark/commit/ba
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19702#discussion_r150381847
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala ---
@@ -281,4 +281,32 @@ class SQLConfSuite extends QueryTest with
Sha
Github user setjet commented on the issue:
https://github.com/apache/spark/pull/18113
An empty sets min and max are defined is -infinity and +infinity:
https://en.wikipedia.org/wiki/Empty_set
This is supported for Java doubles, but not for Longs. We could instead
Long.MIN and Long
Github user setjet commented on a diff in the pull request:
https://github.com/apache/spark/pull/18113#discussion_r150381736
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/typedaggregators.scala
---
@@ -26,43 +26,64 @@ import org.apache.spark.sql.expres
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18113
So the existing typed sum follows the mathmatical standard, not sql
standard. Do we have a mathmatical standard for empty max/min?
---
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18113#discussion_r150381615
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/typedaggregators.scala
---
@@ -26,43 +26,64 @@ import org.apache.spark.sql.exp
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19638#discussion_r150380973
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala
---
@@ -764,13 +764,17 @@ class LinearRegressionSuite
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19713
**[Test build #83721 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83721/testReport)**
for PR 19713 at commit
[`c491974`](https://github.com/apache/spark/commit/c4
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19706
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19713
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19706
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/19706
Thank you @felixcheung and @dongjoon-hyun for review.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user setjet commented on the issue:
https://github.com/apache/spark/pull/18113
@cloud-fan
Sorry i misread the conclusion of the discussion, reverted the initial api
to exactly how it was before, while the new functions follow the SQL standard
as you agreed on 2 weeks ago.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19685
---
-
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/19685
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19713#discussion_r150380582
--- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
@@ -733,7 +733,7 @@ test_that("test cache, uncache and clearCache", {
expect_true(dropTempView
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19721
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user ChenjunZou opened a pull request:
https://github.com/apache/spark/pull/19721
[SPARK-22496][SQL]thrift server adds operation logs
## What changes were proposed in this pull request?
since hive 2.0+ uses upgrade log4j to log4j2ï¼a lot of
[changes](https://issues.apach
Github user mpetruska commented on the issue:
https://github.com/apache/spark/pull/19659
Also, will this work in SparkR, or do I need to add some code/configuration
to expose the functionality to [R]?
---
-
To unsub
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19702#discussion_r150378160
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java
---
@@ -91,11 +92,13 @@
pri
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19702#discussion_r150378929
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala ---
@@ -281,4 +281,32 @@ class SQLConfSuite extends QueryTest with
Sh
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19702#discussion_r150378883
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -285,8 +285,24 @@ object SQLConf {
.booleanConf
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19702#discussion_r150378568
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRowConverter.scala
---
@@ -260,7 +267,6 @@ private[parquet]
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19702#discussion_r150378414
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedParquetRecordReader.java
---
@@ -281,10 +283,11 @@ private
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19690
**[Test build #83720 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83720/testReport)**
for PR 19690 at commit
[`37fba60`](https://github.com/apache/spark/commit/37
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/19690
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19720
**[Test build #83719 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83719/testReport)**
for PR 19720 at commit
[`1722d12`](https://github.com/apache/spark/commit/17
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19690
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/19713
**[Test build #83717 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83717/testReport)**
for PR 19713 at commit
[`c491974`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19713
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83717/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19690
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83716/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19713
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/19690
**[Test build #83716 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83716/testReport)**
for PR 19690 at commit
[`37fba60`](https://github.com/apache/spark/commit/3
101 - 165 of 165 matches
Mail list logo