Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19767
LGTM
---
-
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/19728
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/19728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83983/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19728
**[Test build #83983 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83983/testReport)**
for PR 19728 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
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/19082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83982/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #83982 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83982/testReport)**
for PR 19082 at commit
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/19767#discussion_r151830994
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala
---
@@ -105,6 +105,41 @@ abstract class Expression extends
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/19769#discussion_r151830301
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetInteroperabilitySuite.scala
---
@@ -87,4 +95,109 @@ class
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/19769#discussion_r151830046
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -355,17 +362,33 @@ class
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/19769#discussion_r151830623
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java
---
@@ -430,9 +439,11 @@ private void
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/19769#discussion_r151830543
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java
---
@@ -298,7 +304,10 @@ private void
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19778
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83980/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19778
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/19778
**[Test build #83980 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83980/testReport)**
for PR 19778 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19778
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83979/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19778
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/19778
**[Test build #83979 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83979/testReport)**
for PR 19778 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19777
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83978/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19777
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/19777
**[Test build #83978 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83978/testReport)**
for PR 19777 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r151828392
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -140,6 +140,13 @@ object SQLConf {
.booleanConf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19728
**[Test build #83983 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83983/testReport)**
for PR 19728 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19728
Sure done.
#19777 for `concat_ws`. #19778 for `elt`
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #83982 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83982/testReport)**
for PR 19082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19082
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/19082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83981/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #83981 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83981/testReport)**
for PR 19082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19082
**[Test build #83981 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83981/testReport)**
for PR 19082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19778
**[Test build #83980 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83980/testReport)**
for PR 19778 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19778
**[Test build #83979 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83979/testReport)**
for PR 19778 at commit
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19774#discussion_r151826928
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -689,6 +689,11 @@ case class DescribeColumnCommand(
GitHub user kiszk opened a pull request:
https://github.com/apache/spark/pull/19778
[SPARK-22550][SQL] Fix 64KB JVM bytecode limit problem with elt
## What changes were proposed in this pull request?
This PR changes `elt` code generation to place generated code for
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19777
**[Test build #83978 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83978/testReport)**
for PR 19777 at commit
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/19774
seems good to me. ping @gatorsmile @cloud-fan @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
GitHub user kiszk opened a pull request:
https://github.com/apache/spark/pull/19777
[SPARK-22549][SQL] Fix 64KB JVM bytecode limit problem with concat_ws
## What changes were proposed in this pull request?
This PR changes `concat_ws` code generation to place generated code
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19760
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83977/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19760
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/19760
**[Test build #83977 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83977/testReport)**
for PR 19760 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19776
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user jliwork opened a pull request:
https://github.com/apache/spark/pull/19776
[SPARK-22548][SQL] Incorrect nested AND expression pushed down to JDBâ¦
â¦C data source
## What changes were proposed in this pull request?
Letâs say I have a nested AND
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19390#discussion_r151821865
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -228,24 +254,15 @@ trait
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19772#discussion_r151815120
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/SQLTransformer.scala ---
@@ -70,7 +70,8 @@ class SQLTransformer @Since("1.6.0") (@Since("1.6.0")
Github user zsxwing closed the pull request at:
https://github.com/apache/spark/pull/19771
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/19771
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/19771
Thanks! Merging to master and 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19772#discussion_r151813095
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/SQLTransformer.scala ---
@@ -70,7 +70,8 @@ class SQLTransformer @Since("1.6.0") (@Since("1.6.0")
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19760
**[Test build #83977 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83977/testReport)**
for PR 19760 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19388
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/19388
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83976/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19388
**[Test build #83976 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83976/testReport)**
for PR 19388 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/19763#discussion_r151801786
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -472,17 +474,36 @@ private[spark] class MapOutputTrackerMaster(
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/19763#discussion_r151801570
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -473,16 +477,41 @@ private[spark] class MapOutputTrackerMaster(
}
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19388
High level question: do you need to create the commit id based on the rdd +
stage ids, or could it be something else (like a monotonically increasing
value, or a uuid)? That would probably simplify
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19751
Not retesting since there will probably be feedback and the failure seems
unrelated, so better just wait.
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17436
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83973/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17436
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/17436
**[Test build #83973 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83973/testReport)**
for PR 17436 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19751
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/19751
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83975/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19751
**[Test build #83975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83975/testReport)**
for PR 19751 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19728
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83974/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19728
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/19728
**[Test build #83974 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83974/testReport)**
for PR 19728 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19388
**[Test build #83976 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83976/testReport)**
for PR 19388 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17436
LGTM except a few minor comment, please update the PR title and
description, thanks!
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r151765263
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetReadBenchmark.scala
---
@@ -260,6 +261,7 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r151764870
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -62,7 +69,11 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r151764498
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedParquetRecordReader.java
---
@@ -101,9 +101,13 @@
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r151764317
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -140,6 +140,13 @@ object SQLConf {
.booleanConf
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19728
can you split it into 3 PRs? The approaches for these 3 expression are
quite different.
---
-
To unsubscribe, e-mail:
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/19775
@matyix thanks for re-submitting!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19751
**[Test build #83975 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83975/testReport)**
for PR 19751 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19751
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/19774
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/19774
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83972/
Test PASSed.
---
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19760
> my future plan is to move config related stuff to a new maven module
Sounds good. I'll try to update this before leaving on vacation, but may
not be able to...
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19774
**[Test build #83972 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83972/testReport)**
for PR 19774 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19257
Thank you for the decision, @cloud-fan . It's great to see the progress on
this!
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19769
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/19769
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83971/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19769
**[Test build #83971 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83971/testReport)**
for PR 19769 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19728
**[Test build #83974 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83974/testReport)**
for PR 19728 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17436
**[Test build #83973 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83973/testReport)**
for PR 17436 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19730
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83970/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19730
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/19730
**[Test build #83970 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83970/testReport)**
for PR 19730 at commit
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/19725#discussion_r151741374
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/InjectPlaceholderExchange.scala
---
@@ -0,0 +1,69 @@
+/*
+ *
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19717
@vanzin @jiangxb1987 @mridulm PTAL. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19774#discussion_r151741011
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -689,6 +689,11 @@ case class DescribeColumnCommand(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19728#discussion_r151741098
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -125,19 +138,43 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19728#discussion_r151740628
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -125,19 +138,43 @@ case class
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/19773#discussion_r151739773
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -318,16 +318,28 @@ case class
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/19773#discussion_r151740225
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -1459,6 +1459,11 @@ abstract class DDLSuite extends
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/19773#discussion_r151739604
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -318,16 +318,28 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19728#discussion_r151738992
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -63,15 +63,28 @@ case class
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/19774#discussion_r151737674
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -689,6 +689,11 @@ case class DescribeColumnCommand(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19747
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83969/
Test PASSed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19772
---
-
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/19747
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
1 - 100 of 212 matches
Mail list logo