Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18655
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18655
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79786/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18655
**[Test build #79786 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79786/testReport)**
for PR 18655 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18649
I'm not familiar with this code and don't know who I should ping, cc
@srowen do we have a maintainer for thrift-server?
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18651
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79789/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18651
**[Test build #79789 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79789/testReport)**
for PR 18651 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18668
ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128449351
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java
---
@@ -0,0 +1,545 @@
+/*
+ * Licensed to the
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18682
Merged to master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18682
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128449421
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java
---
@@ -0,0 +1,545 @@
+/*
+ * Licensed to the
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18683
I'm not convinced we need this configuration, we can always control the
size of each fetch request by `maxSizeInFlights`, so why bother importing this
new config?
---
If your project is set
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18468#discussion_r128457753
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/vectorized/ColumnarBatchSuite.scala
---
@@ -326,6 +329,80 @@ class ColumnarBatchSuite
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r128439050
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -753,17 +753,26 @@ class AstBuilder(conf: SQLConf)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12646
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/18687
[SPARK-21484][SQL] Fix inconsistent query plans of Dataset after
persist/unpersist
## What changes were proposed in this pull request?
After the call of persist/unpersis, the query plans
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17848#discussion_r128444612
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/UDFRegistration.scala ---
@@ -130,460 +138,507 @@ class UDFRegistration private[sql]
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17848#discussion_r128443096
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/UDFRegistration.scala ---
@@ -79,8 +79,15 @@ class UDFRegistration private[sql] (functionRegistry:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17848#discussion_r12832
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/UDFRegistration.scala ---
@@ -130,460 +138,507 @@ class UDFRegistration private[sql]
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17848#discussion_r128444945
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -3185,170 +3185,207 @@ object functions {
val inputTypes = (1 to
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18492#discussion_r128452352
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -572,20 +572,35 @@ private[spark] class
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r128456263
--- Diff: python/pyspark/ml/tests.py ---
@@ -1389,11 +1389,25 @@ def test_output_columns(self):
(2.0,
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r128455590
--- Diff: python/pyspark/ml/classification.py ---
@@ -1560,8 +1581,9 @@ def trainSingleClass(index):
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r128456402
--- Diff: python/pyspark/ml/tests.py ---
@@ -1389,11 +1389,25 @@ def test_output_columns(self):
(2.0,
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r128453820
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/OneVsRest.scala ---
@@ -294,6 +296,18 @@ final class OneVsRest @Since("1.4.0") (
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r128455001
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/OneVsRestSuite.scala ---
@@ -101,6 +101,45 @@ class OneVsRestSuite extends SparkFunSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18688
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user djvulee commented on a diff in the pull request:
https://github.com/apache/spark/pull/18651#discussion_r128435807
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -525,8 +535,9 @@ private[yarn] class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18503#discussion_r128439416
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala
---
@@ -363,7 +363,8 @@
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128448056
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java
---
@@ -0,0 +1,545 @@
+/*
+ * Licensed to the
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18684#discussion_r128449639
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/BypassMergeSortShuffleWriter.java
---
@@ -188,17 +189,20 @@ public void
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18684#discussion_r128447912
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/buffer/FileSegmentManagedBuffer.java
---
@@ -132,7 +134,8 @@ public Object
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18684#discussion_r128448743
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/BypassMergeSortShuffleWriter.java
---
@@ -188,17 +189,20 @@ public void
GitHub user wangyangting opened a pull request:
https://github.com/apache/spark/pull/18688
delete the superfluous â symbol
**'ByKey** --> **ByKey**
and `'ByKey` operations --> and `ByKey` operations
## How was this patch tested?
before -->
![5x9 z l
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/17419
As the dataframe version is much slower than RDD version (currently test
against vector of size 1)
I also guess there is some performance issue in
Github user debugger87 commented on the issue:
https://github.com/apache/spark/pull/18649
@cloud-fan This patch is very simple and clear, is there any problem to
review or merge it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18651
**[Test build #79789 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79789/testReport)**
for PR 18651 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18680
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r128443990
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -753,17 +753,26 @@ class AstBuilder(conf: SQLConf)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18353
**[Test build #3846 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3846/testReport)**
for PR 18353 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18668
**[Test build #79792 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79792/testReport)**
for PR 18668 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128451896
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java
---
@@ -0,0 +1,545 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18687
**[Test build #79791 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79791/testReport)**
for PR 18687 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18680
@BryanCutler all classes under the `execution` package are meant to be
private, in the future we will move them to a new package if we are ready to
public them.
---
If your project is set up
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r128429389
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/stat/SummarizerSuite.scala ---
@@ -0,0 +1,406 @@
+/*
+ * Licensed to the Apache Software
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r128428434
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -0,0 +1,746 @@
+/*
+ * Licensed to the Apache Software
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r128429254
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -0,0 +1,799 @@
+/*
+ * Licensed to the Apache Software
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r128428604
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -0,0 +1,746 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79787/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18680
**[Test build #79787 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79787/testReport)**
for PR 18680 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18651
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12646
**[Test build #79788 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79788/testReport)**
for PR 12646 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12646
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79788/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r128438692
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -753,17 +753,26 @@ class AstBuilder(conf: SQLConf)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18687
**[Test build #79790 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79790/testReport)**
for PR 18687 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17848#discussion_r128444145
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/UDFRegistration.scala ---
@@ -130,460 +138,507 @@ class UDFRegistration private[sql]
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r128453437
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -753,17 +753,26 @@ class AstBuilder(conf: SQLConf)
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128477508
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java
---
@@ -0,0 +1,545 @@
+/*
+ * Licensed to the
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18388#discussion_r128485815
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/util/TransportConf.java
---
@@ -257,4 +257,7 @@ public Properties cryptoConf() {
Github user mdespriee commented on the issue:
https://github.com/apache/spark/pull/17461
Hi @hhbyyh, @jkbradley
a gentle ping on this PR, if you could have a look at the code, and give me
your opinion regarding my question hereabove (the use of Param API to provide a
previous
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18323
**[Test build #79794 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79794/testReport)**
for PR 18323 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18323
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18323
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79794/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18323
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16924
Is there any concern regarding checking file length? I guess we have no
much choices here to detect file changes?
There are comments for why using file length to detect file changes:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18680
thanks, merging to master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18323
**[Test build #79799 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79799/testReport)**
for PR 18323 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18388#discussion_r128526865
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/server/TransportRequestHandler.java
---
@@ -145,7 +172,12 @@ private void
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18388#discussion_r128502814
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/server/TransportRequestHandler.java
---
@@ -145,7 +172,12 @@ private void
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18689
**[Test build #79797 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79797/testReport)**
for PR 18689 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18655
**[Test build #79798 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79798/testReport)**
for PR 18655 at commit
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18388#discussion_r128519475
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/server/TransportRequestHandler.java
---
@@ -145,7 +172,12 @@ private void
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18353
**[Test build #3847 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3847/testReport)**
for PR 18353 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18323
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79795/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18323
**[Test build #79795 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79795/testReport)**
for PR 18323 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16924
ok then this LGTM, but I'm not very confident about this part of code, cc
@vanzin
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18388
**[Test build #79800 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79800/testReport)**
for PR 18388 at commit
Github user djvulee commented on a diff in the pull request:
https://github.com/apache/spark/pull/18651#discussion_r128432387
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -242,7 +244,7 @@ private[yarn] class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18503#discussion_r128438054
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamSuite.scala ---
@@ -479,6 +479,61 @@ class StreamSuite extends StreamTest {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17848#discussion_r128439889
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/expressions/UserDefinedFunction.scala
---
@@ -84,22 +90,38 @@ case class UserDefinedFunction
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17848#discussion_r128439969
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/UDFRegistration.scala ---
@@ -79,8 +79,15 @@ class UDFRegistration private[sql] (functionRegistry:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128446137
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ReadOnlyColumnVector.java
---
@@ -0,0 +1,250 @@
+/*
+ * Licensed to
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128449912
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java
---
@@ -0,0 +1,545 @@
+/*
+ * Licensed to the
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18492#discussion_r128452499
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -572,20 +572,35 @@ private[spark] class
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18305#discussion_r128458132
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/optim/aggregator/LogisticAggregatorSuite.scala
---
@@ -0,0 +1,254 @@
+/*
+ * Licensed to the
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18680#discussion_r128503537
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ArrowColumnVector.java
---
@@ -0,0 +1,545 @@
+/*
+ * Licensed to the
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18680
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18680
**[Test build #79793 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79793/testReport)**
for PR 18680 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18388#discussion_r128503793
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/util/TransportConf.java
---
@@ -257,4 +257,7 @@ public Properties cryptoConf() {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79793/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18680
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18468#discussion_r128507888
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/CachedBatchColumnVector.java
---
@@ -0,0 +1,417 @@
+/*
+ * Licensed
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18388
a question: can we track the number of sending chunks in
`TransportRequestHandler` instead of `StreamManager`? It looks weird to me that
only `OneForOneStreamManager` does the tracking while we
Github user baibaichen commented on the issue:
https://github.com/apache/spark/pull/18652
@viirya , @jiangxb1987 @gatorsmile
In general, Hive doesn't consider non-deterministic in join condition.
Some terms:
1 equi-joins with key, i.e. a.key = b.key, using
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18323
**[Test build #79794 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79794/testReport)**
for PR 18323 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18680
LGTM, pending jenkins
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18681
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18388#discussion_r128498015
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/util/TransportConf.java
---
@@ -257,4 +257,7 @@ public Properties cryptoConf() {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18687
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79790/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18687
**[Test build #79790 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79790/testReport)**
for PR 18687 at commit
1 - 100 of 322 matches
Mail list logo