Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88387/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
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/20847
**[Test build #88387 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88387/testReport)**
for PR 20847 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20829
**[Test build #88395 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88395/testReport)**
for PR 20829 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20807
---
-
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/20579
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/20579
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88388/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20579
**[Test build #88388 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88388/testReport)**
for PR 20579 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20807
Merging to master / 2.3.
---
-
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/20829
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/20853
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/20829
**[Test build #88392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88392/testReport)**
for PR 20829 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r175629947
--- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala ---
@@ -126,7 +126,11 @@ private[spark] abstract class WebUI(
def bind(): Unit = {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20829
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88392/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20853
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88385/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20853
**[Test build #88385 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88385/testReport)**
for PR 20853 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/20657
I'm really sorry about the delay @vanzin @squito . I will take another
review today and back to you.
---
-
To unsubscribe,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20796#discussion_r175626064
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -187,8 +218,9 @@ public void writeTo(OutputStream out) throws
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20433#discussion_r175625855
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -616,18 +616,22 @@ booleanValue
;
interval
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20829
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88390/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20829
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/20829
**[Test build #88390 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88390/testReport)**
for PR 20829 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20796#discussion_r175624764
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -57,12 +57,39 @@
public Object getBaseObject() { return
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20796
**[Test build #88394 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88394/testReport)**
for PR 20796 at commit
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20433#discussion_r175624411
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1479,15 +1479,13 @@ class SQLQuerySuite extends QueryTest with
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20796
retest 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/20796
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88384/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20796
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/20856
BTW, I've just realized that even without the issue it's clear that
creating a new `FileSourceScanExec` will end up with a NPE from the
`supportsBatch` field.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20796
**[Test build #88384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88384/testReport)**
for PR 20796 at commit
Github user gerashegalov commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r175623955
--- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala ---
@@ -126,7 +126,11 @@ private[spark] abstract class WebUI(
def bind(): Unit = {
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/20856
I spent over 2 days applying different modifications to the query hoping I
could cut the number of `CASE WHEN`s and other projections, but noticed no
correlation between the number or their
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20433#discussion_r175622969
--- Diff: sql/core/src/test/resources/sql-tests/inputs/interval.sql ---
@@ -0,0 +1,184 @@
+-- Basic tests for intervals
+
+select
+ '1'
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20827
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/1620/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20827
**[Test build #88393 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88393/testReport)**
for PR 20827 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20827
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20827#discussion_r175621271
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
---
@@ -324,31 +324,28 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20829
**[Test build #88392 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88392/testReport)**
for PR 20829 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20687
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/1619/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20687
**[Test build #88391 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88391/testReport)**
for PR 20687 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20687
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user henryr commented on the issue:
https://github.com/apache/spark/pull/20687
@gatorsmile ok, I think the coverage right now is a reasonable start - the
other test cases I can think of would act more like they're exercising the
expression-walking code, not the actual
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20795
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/20829
**[Test build #88390 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88390/testReport)**
for PR 20829 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20795
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88386/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20853
**[Test build #88389 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88389/testReport)**
for PR 20853 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20795
**[Test build #88386 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88386/testReport)**
for PR 20795 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20858
Thinks for this work! One question; why do you think we need to support
this api in Spark native? Other libraries support this as first-class?
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20850
@hvanhovell btw, (this is not related to this pr thought...) the most part
of code in `UTF8StringBuffer` and `BufferHolder` are overlapped. So, we could
clean up there, too?
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20803#discussion_r175609377
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -166,20 +168,28 @@ private[sql] object Dataset {
class Dataset[T]
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20850#discussion_r175609234
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeRowWriter.java
---
@@ -40,29 +37,45 @@
*/
public
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20803#discussion_r175608866
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -166,20 +168,28 @@ private[sql] object Dataset {
class Dataset[T]
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20850#discussion_r175608496
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/BufferHolder.java
---
@@ -86,11 +88,39 @@ public void grow(int
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20806
great! maybe we can hold this PR for a real SQL tree aggregate in the
future, with some proper design and discussion.
---
-
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20850#discussion_r175608099
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeWriter.java
---
@@ -17,17 +17,71 @@
package
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20701
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/20579
**[Test build #88388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88388/testReport)**
for PR 20579 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20701
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88381/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20579
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/1618/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20579
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/20701
**[Test build #88381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88381/testReport)**
for PR 20701 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19222
Looks pretty good! My major concern is, the semantic of offset in
`MemoryBlock.getXXX` should be implementation independent. Users should always
assume the offset starts with 0. Since it's about
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175605407
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -119,15 +115,24 @@ public static UTF8String blankString(int
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175605220
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -119,15 +115,24 @@ public static UTF8String blankString(int
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175605033
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -119,15 +115,24 @@ public static UTF8String blankString(int
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175604820
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/UnsafeMemoryAllocator.java
---
@@ -36,22 +42,34 @@ public MemoryBlock allocate(long
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175604535
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -45,38 +45,159 @@
*/
public static final int
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175604062
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/HeapMemoryAllocator.java
---
@@ -94,12 +95,12 @@ public void free(MemoryBlock
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20796
LGTM, pending jenkins
---
-
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/20847
**[Test build #88387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88387/testReport)**
for PR 20847 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
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/1617/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20847
Retest 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/20847
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88382/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20847
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/20847
**[Test build #88382 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88382/testReport)**
for PR 20847 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20795
**[Test build #88386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88386/testReport)**
for PR 20795 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175592083
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/hash/Murmur3_x86_32.java ---
@@ -49,55 +51,81 @@ public static int hashInt(int input, int
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
Which document do you have in mind? A section in
`docs/sql-programming-guide.md`?
---
-
To unsubscribe, e-mail:
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20796#discussion_r175590297
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -57,12 +57,39 @@
public Object getBaseObject() { return
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20796#discussion_r175589638
--- Diff:
common/unsafe/src/test/java/org/apache/spark/unsafe/types/UTF8StringSuite.java
---
@@ -791,4 +795,21 @@ public void trimRightWithTrimString() {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20859
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88383/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20859
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/20859
**[Test build #88383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88383/testReport)**
for PR 20859 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20818
**[Test build #4141 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4141/testReport)**
for PR 20818 at commit
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20859#discussion_r175585873
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationChecker.scala
---
@@ -160,6 +160,19 @@ object
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20853#discussion_r175585634
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -137,16 +138,36 @@ private[deploy] object DependencyUtils {
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20853#discussion_r175585581
--- Diff: core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
---
@@ -137,16 +138,36 @@ private[deploy] object DependencyUtils {
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20809
> This is on travis and no SPARK_HOME is set.
That sounds a little odd. If that is true, then your proposed code wouldn't
work either, since it requires SPARK_HOME to be known.
In
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20850
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/20850
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88380/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20853
**[Test build #88385 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88385/testReport)**
for PR 20853 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20850
**[Test build #88380 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88380/testReport)**
for PR 20850 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20208
Let us write the document in this PR? Document our current behaviors with
the related test cases.
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20853
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/20853
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88379/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20853
**[Test build #88379 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88379/testReport)**
for PR 20853 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r175581710
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/array/ByteArrayMethods.java
---
@@ -48,6 +49,16 @@ public static int
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20796
**[Test build #88384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88384/testReport)**
for PR 20796 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20816#discussion_r175580746
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -669,11 +672,42 @@ object
101 - 200 of 479 matches
Mail list logo
