Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19438
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/19438
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82534/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19438
**[Test build #82534 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82534/testReport)**
for PR 19438 at commit
[`49262d1`](https://github.com/apache/spark/commit/4
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19453
This sounds identical to `drop`?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19453
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user sohum2002 opened a pull request:
https://github.com/apache/spark/pull/19453
Added selectAllColumns function in Dataset class
The proposed two new additional functions is to help select all the columns
in a Dataset except for given columns.
You can merge this pull reques
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19452
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/19452
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82533/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19452
**[Test build #82533 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82533/testReport)**
for PR 19452 at commit
[`8c2a39f`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19438
**[Test build #82534 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82534/testReport)**
for PR 19438 at commit
[`49262d1`](https://github.com/apache/spark/commit/49
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19438#discussion_r143321930
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/QuantileSummariesSuite.scala
---
@@ -58,7 +58,7 @@ class QuantileSummariesSuite exte
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/19294
Thanks for the fix @szhem, great work !
Merged to master and 2.2.1
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.ap
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19294
---
-
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/19394
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/19394
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82532/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19394
**[Test build #82532 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82532/testReport)**
for PR 19394 at commit
[`56089f5`](https://github.com/apache/spark/commit/5
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19394#discussion_r143321383
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -274,19 +274,26 @@ abstract class SparkPlan extends QueryPlan[SparkPl
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r143321347
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/WholeStageCodegenSuite.scala
---
@@ -181,7 +181,7 @@ class WholeStageCodegenSuite extends S
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18931
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19394#discussion_r143321349
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -274,19 +274,26 @@ abstract class SparkPlan extends QueryPlan[SparkPl
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18931
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82531/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18931
**[Test build #82531 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82531/testReport)**
for PR 18931 at commit
[`601c225`](https://github.com/apache/spark/commit/6
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r143321131
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/WholeStageCodegenSuite.scala
---
@@ -181,7 +181,7 @@ class WholeStageCodegenSuite extends S
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19452
**[Test build #82533 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82533/testReport)**
for PR 19452 at commit
[`8c2a39f`](https://github.com/apache/spark/commit/8c
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19447
@gatorsmile I am happy to take it. We have [one test
case](https://github.com/apache/spark/pull/16648/files#diff-c7f041fda7fd9aa1a5225f86bab4b1b0R69)
that cause the error `JVM limit of 0x`. I am n
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19447
How about making a separated function for checking the threshold and then
test it like #18810:
https://github.com/apache/spark/pull/18810/files#diff-8bcc5aea39c73d4bf38aef6f6951d42cR363
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19294
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/19294
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82529/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19294
**[Test build #82529 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82529/testReport)**
for PR 19294 at commit
[`f55b7c2`](https://github.com/apache/spark/commit/f
Github user minixalpha commented on the issue:
https://github.com/apache/spark/pull/19090
Thanks, @HyukjinKwon @jsnowacki @felixcheung
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19394
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82530/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19394
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/19394
**[Test build #82530 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82530/testReport)**
for PR 19394 at commit
[`5b92fe2`](https://github.com/apache/spark/commit/5
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19449#discussion_r143318406
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -929,7 +929,7 @@ class CodegenContext {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19394
**[Test build #82532 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82532/testReport)**
for PR 19394 at commit
[`56089f5`](https://github.com/apache/spark/commit/56
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/19394#discussion_r143317742
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/ConfigBehaviorSuite.scala ---
@@ -58,7 +58,7 @@ class ConfigBehaviorSuite extends QueryTest with
Shar
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/19394#discussion_r143317686
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -280,13 +280,20 @@ abstract class SparkPlan extends QueryPlan[SparkPl
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18931
**[Test build #82531 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82531/testReport)**
for PR 18931 at commit
[`601c225`](https://github.com/apache/spark/commit/60
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/19394#discussion_r143317518
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -280,13 +280,20 @@ abstract class SparkPlan extends QueryPlan[SparkPl
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19082
Sure. I'm totally agreed. We need to know the advantages and possible
impacts if any when merging this PR and #18931,. It is good @kiszk and
@rednaxelafx can help review this PR and #18931.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19452
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82528/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19452
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/19452
**[Test build #82528 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82528/testReport)**
for PR 19452 at commit
[`026a33b`](https://github.com/apache/spark/commit/0
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/19448
+1
I completely agree that using a ParquetOutputCommitter should be optional.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19394#discussion_r143315153
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -280,13 +280,20 @@ abstract class SparkPlan extends QueryPlan[Spa
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19394#discussion_r143315127
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -280,13 +280,20 @@ abstract class SparkPlan extends QueryPlan[Spa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18460
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/18460
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82527/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18460
**[Test build #82527 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82527/testReport)**
for PR 18460 at commit
[`67a037c`](https://github.com/apache/spark/commit/6
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18732#discussion_r143313284
--- Diff: python/pyspark/sql/group.py ---
@@ -192,7 +193,69 @@ def pivot(self, pivot_col, values=None):
jgd = self._jgd.pivot(pivot_col)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19452
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82526/
Test PASSed.
---
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/19437#discussion_r143312903
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackendUtil.scala
---
@@ -17,10 +17,14 @@
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/19437#discussion_r143312857
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackendUtil.scala
---
@@ -122,7 +126,7 @@ private[me
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19394
**[Test build #82530 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82530/testReport)**
for PR 19394 at commit
[`5b92fe2`](https://github.com/apache/spark/commit/5b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19452
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19394
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/19449#discussion_r143312237
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -929,7 +929,7 @@ class CodegenContex
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19452
**[Test build #82526 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82526/testReport)**
for PR 19452 at commit
[`c90fb3c`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18460
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/18460
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82523/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18460
**[Test build #82523 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82523/testReport)**
for PR 18460 at commit
[`c72aa18`](https://github.com/apache/spark/commit/c
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/19394
Anyone have a clue what the python error could be? It doesn't look related.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spar
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19449#discussion_r143307261
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -929,7 +929,7 @@ class CodegenContext
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19267
Sorry I haven't been having time to look at anything around here lately. I
recommend you make some noise in the mailing list or ping other people if you
want to get attention to this.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19449#discussion_r143306861
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -929,7 +929,7 @@ class CodegenContext
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19394
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82524/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19394
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/19394
**[Test build #82524 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82524/testReport)**
for PR 19394 at commit
[`f6613c2`](https://github.com/apache/spark/commit/f
Github user juanrh commented on the issue:
https://github.com/apache/spark/pull/19267
Hi @vanzin, do you have any comments on the design document attached above?
Thanks
---
-
To unsubscribe, e-mail: reviews
Github user szhem commented on a diff in the pull request:
https://github.com/apache/spark/pull/19294#discussion_r143306215
--- Diff:
core/src/main/scala/org/apache/spark/internal/io/HadoopMapReduceCommitProtocol.scala
---
@@ -57,6 +60,15 @@ class HadoopMapReduceCommitProtocol(job
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19294
**[Test build #82529 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82529/testReport)**
for PR 19294 at commit
[`f55b7c2`](https://github.com/apache/spark/commit/f5
Github user szhem commented on a diff in the pull request:
https://github.com/apache/spark/pull/19294#discussion_r143305853
--- Diff:
core/src/main/scala/org/apache/spark/internal/io/HadoopMapReduceCommitProtocol.scala
---
@@ -57,6 +60,15 @@ class HadoopMapReduceCommitProtocol(job
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19270
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82522/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19270
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/19270
**[Test build #82522 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82522/testReport)**
for PR 19270 at commit
[`25b5215`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19450
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82521/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19450
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/19450
**[Test build #82521 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82521/testReport)**
for PR 19450 at commit
[`5a9ef13`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19394
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/19394
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82525/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19394
**[Test build #82525 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82525/testReport)**
for PR 19394 at commit
[`5b92fe2`](https://github.com/apache/spark/commit/5
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/18664
Made [SPARK-1](https://issues.apache.org/jira/browse/SPARK-1) for
user doc, once we decide what to do with timestampes it can be completed
---
-
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/18664
Bryan, I haven't created. Go ahead!
On Fri, Oct 6, 2017 at 5:45 PM Bryan Cutler
wrote:
> Thanks all for the discussion. I think there are a lot of subtleties at
> play here, a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19452
**[Test build #82528 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82528/testReport)**
for PR 19452 at commit
[`026a33b`](https://github.com/apache/spark/commit/02
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19372
**[Test build #3943 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3943/testReport)**
for PR 19372 at commit
[`2ea3f18`](https://github.com/apache/spark/commit/
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/18664
Thanks all for the discussion. I think there are a lot of subtleties at
play here, and what may or may not be considered a bug can depend on the users
intent. Regardless, I agree that there need
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18460
**[Test build #82527 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82527/testReport)**
for PR 18460 at commit
[`67a037c`](https://github.com/apache/spark/commit/67
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/18460#discussion_r143295197
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -100,6 +101,17 @@ object TypeCoercion {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19450#discussion_r143294931
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/sasl/ShuffleSecretManagerSuite.java
---
@@ -0,0 +1,55 @@
+/*
+ * Licensed to
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/19449
LGTM. There are already multiple places in codegen where `SQLConf.get` is
being used so this will make things consistent
---
---
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/19449#discussion_r143291724
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -929,7 +929,7 @@ class CodegenContex
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19452
**[Test build #82526 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82526/testReport)**
for PR 19452 at commit
[`c90fb3c`](https://github.com/apache/spark/commit/c9
GitHub user joseph-torres opened a pull request:
https://github.com/apache/spark/pull/19452
[SPARK-22136][SS] Evaluate one-sided conditions early in stream-stream
joins.
## What changes were proposed in this pull request?
Evaluate one-sided conditions early in stream-stream
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19449
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/19449
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82520/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19449
**[Test build #82520 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82520/testReport)**
for PR 19449 at commit
[`be4220d`](https://github.com/apache/spark/commit/b
Github user sathiyapk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19451#discussion_r143289752
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1242,6 +1243,53 @@ object ReplaceIntersectWithSemiJ
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/18460#discussion_r143289255
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -100,6 +101,17 @@ object TypeCoercion {
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/18460#discussion_r143288998
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -100,6 +101,17 @@ object TypeCoercion {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18460#discussion_r143288861
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -100,6 +101,17 @@ object TypeCoercion {
1 - 100 of 357 matches
Mail list logo