Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
**[Test build #88223 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88223/testReport)**
for PR 20659 at commit
[`f478c89`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88223/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #88223 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88223/testReport)**
for PR 20659 at commit
[`f478c89`](https://github.com/apache/spark/commit/f4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
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/1501/
Tes
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20659
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20659
> [error]
/home/jenkins/workspace/SparkPullRequestBuilder/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionImpl.java:825:
error: cannot find symbol
[error]
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174361814
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/array/LongArray.java ---
@@ -80,7 +76,7 @@ public void zeroOut() {
public void set(int in
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20806#discussion_r174358970
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1658,6 +1659,43 @@ class Dataset[T] private[sql](
def groupByKey[K](func:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20817
**[Test build #88222 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88222/testReport)**
for PR 20817 at commit
[`b7bce25`](https://github.com/apache/spark/commit/b7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20817
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/20817
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/1500/
Tes
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20817
cc @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/20817
[SPARK-23599][SQL] Add a UUID generator from Pseudo-Random Numbers
## What changes were proposed in this pull request?
This patch adds a UUID generator from Pseudo-Random Numbers. We can use
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #88221 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88221/testReport)**
for PR 20659 at commit
[`f478c89`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88221/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
**[Test build #88221 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88221/testReport)**
for PR 20659 at commit
[`f478c89`](https://github.com/apache/spark/commit/f4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/1499/
Tes
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20727
@cloud-fan and @MaxGekk, I believe this is ready for another look.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.ap
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20796#discussion_r174355311
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1279,4 +1279,22 @@ class CSVSuite extends Query
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20796
LGTM for the fix. +1 for more tests.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20579
I think this should be applied to all data sources not only parquet. I
can't think of any cases that a data source needs to write data with empty
schema, cc @rdblue for confirmation.
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20790
**[Test build #88220 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88220/testReport)**
for PR 20790 at commit
[`6656be7`](https://github.com/apache/spark/commit/66
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20790
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174350660
--- Diff: bin/docker-image-tool.sh ---
@@ -63,9 +63,11 @@ function build {
error "Cannot find docker image. This script must be run from a
runna
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20806#discussion_r174350193
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1658,6 +1659,43 @@ class Dataset[T] private[sql](
def groupByKey[K](fun
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174348342
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/submit/ClientSuite.scala
---
@@ -108,62 +111,42 @@ class ClientSu
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174345430
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/KubernetesUtils.scala
---
@@ -89,26 +56,16 @@ private[spark] obje
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174345374
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/KubernetesUtils.scala
---
@@ -89,26 +56,16 @@ private[spark] obje
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174345265
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesClientApplication.scala
---
@@ -180,6 +179,26 @@
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20806#discussion_r174337702
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1658,6 +1659,43 @@ class Dataset[T] private[sql](
def groupByKey[K](func:
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/20690
@jiangxb1987 @jerryshao Could you help review it ? thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20816
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/20816
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88216/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20816
**[Test build #88216 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88216/testReport)**
for PR 20816 at commit
[`1c5dedb`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20799
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88219/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20799
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/20799
**[Test build #88219 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88219/testReport)**
for PR 20799 at commit
[`2afc91e`](https://github.com/apache/spark/commit/2
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20806#discussion_r174336159
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1658,6 +1659,43 @@ class Dataset[T] private[sql](
def groupByKey[K](func:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20799
**[Test build #88219 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88219/testReport)**
for PR 20799 at commit
[`2afc91e`](https://github.com/apache/spark/commit/2a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20799
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/20799
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/1498/
Tes
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174334386
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/hash/Murmur3_x86_32.java ---
@@ -49,55 +50,81 @@ public static int hashInt(int input, int seed)
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/20812
@vanzin @zsxwing @jerryshao
How do you think about this ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174333828
--- Diff:
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/expressions/HiveHasher.java
---
@@ -38,12 +39,18 @@ public static int hashLong(long input
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174333675
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/ByteArrayMemoryBlock.java
---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apache S
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88218/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #88218 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88218/testReport)**
for PR 20659 at commit
[`b418909`](https://github.com/apache/spark/commit/b
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20802#discussion_r174332968
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -975,6 +975,8 @@ private[spark] object Utils extends Logging {
def check
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #88218 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88218/testReport)**
for PR 20659 at commit
[`b418909`](https://github.com/apache/spark/commit/b4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/1497/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20659
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88217/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #88217 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88217/testReport)**
for PR 20659 at commit
[`b418909`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20799#discussion_r174331279
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ExecutorRunnable.scala
---
@@ -247,6 +241,18 @@ private[yarn] class Executor
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #88217 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88217/testReport)**
for PR 20659 at commit
[`b418909`](https://github.com/apache/spark/commit/b4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
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/1496/
Tes
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174329806
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/submit/ClientSuite.scala
---
@@ -108,62 +111,42 @@ class ClientSuite
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174329513
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/KubernetesUtils.scala
---
@@ -89,26 +56,16 @@ private[spark] object
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174329568
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/KubernetesUtils.scala
---
@@ -89,26 +56,16 @@ private[spark] object
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174328774
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesClientApplication.scala
---
@@ -180,6 +179,26 @@ pr
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174328964
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesClientApplication.scala
---
@@ -202,6 +221,10 @@ pr
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/20669#discussion_r174328623
--- Diff: bin/docker-image-tool.sh ---
@@ -63,9 +63,11 @@ function build {
error "Cannot find docker image. This script must be run from a
runnable
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20816
**[Test build #88216 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88216/testReport)**
for PR 20816 at commit
[`1c5dedb`](https://github.com/apache/spark/commit/1c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20816
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/1495/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20816
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user susanxhuynh commented on the issue:
https://github.com/apache/spark/pull/20640
@skonto Yes, I'm ok with that. Sorry for the delayed response.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20816
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88215/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20816
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/20816
**[Test build #88215 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88215/testReport)**
for PR 20816 at commit
[`ac17976`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20815
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88213/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20815
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/20815
**[Test build #88213 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88213/testReport)**
for PR 20815 at commit
[`1518a5a`](https://github.com/apache/spark/commit/1
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20796
the fix LGTM, we can add more tests for different ranges of the invalid
chars.
---
-
To unsubscribe, e-mail: reviews-unsubscr.
Github user maryannxue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r174303151
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1071,6 +1072,66 @@ object PushPredicateThroughJoin
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r174302717
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1071,6 +1072,66 @@ object PushPredicateThroughJoin
Github user maryannxue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r174302419
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1071,6 +1072,66 @@ object PushPredicateThroughJoin
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r174301967
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/InferFiltersFromConstraintsSuite.scala
---
@@ -204,4 +205,40 @@ class Infe
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174295581
--- Diff:
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/expressions/HiveHasher.java
---
@@ -38,12 +39,18 @@ public static int hashLong(long i
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174300387
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/hash/Murmur3_x86_32.java ---
@@ -49,55 +50,81 @@ public static int hashInt(int input, int se
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174300982
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/OffHeapMemoryBlock.java
---
@@ -0,0 +1,108 @@
+/*
+ * Licensed to the Apache
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174301203
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/ByteArrayMemoryBlock.java
---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apac
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174295402
--- Diff:
common/unsafe/src/main/java/org/apache/spark/sql/catalyst/expressions/HiveHasher.java
---
@@ -38,12 +39,18 @@ public static int hashLong(long i
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r174296632
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/array/LongArray.java ---
@@ -80,7 +76,7 @@ public void zeroOut() {
public void set(in
Github user maryannxue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r174300996
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/InferFiltersFromConstraintsSuite.scala
---
@@ -204,4 +205,40 @@ class Infe
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20779
---
-
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/20816#discussion_r174299561
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/InferFiltersFromConstraintsSuite.scala
---
@@ -204,4 +205,40 @@ class Infe
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r174299321
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1071,6 +1072,66 @@ object PushPredicateThroughJoin
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20814
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88211/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20814
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/20814
**[Test build #88211 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88211/testReport)**
for PR 20814 at commit
[`a897277`](https://github.com/apache/spark/commit/a
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r174297591
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1071,6 +1072,66 @@ object PushPredicateThroughJoin
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20779
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
1 - 100 of 282 matches
Mail list logo
