Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14007
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14007
**[Test build #61625 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61625/consoleFull)**
for PR 14007 at commit
[`bd83643`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13704
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61629/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13704
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13704
**[Test build #61629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61629/consoleFull)**
for PR 13704 at commit
[`2d3d34a`](https://github.com/apache/spark/commit/
Github user drcrallen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69344006
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSched
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61628/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13981
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14016
**[Test build #61633 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61633/consoleFull)**
for PR 14016 at commit
[`871281d`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13981
**[Test build #61628 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61628/consoleFull)**
for PR 13981 at commit
[`04bdb3a`](https://github.com/apache/spark/commit/
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69343465
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -652,6 +656,129 @@ case class StringRPa
Github user MechCoder commented on the issue:
https://github.com/apache/spark/pull/14016
@srowen fixed.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14016
**[Test build #61632 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61632/consoleFull)**
for PR 14016 at commit
[`01c405b`](https://github.com/apache/spark/commit/0
Github user drcrallen commented on the issue:
https://github.com/apache/spark/pull/11157
How would I set a JMX port or debug port with this approach?
---
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 no
Github user tmagrino commented on the issue:
https://github.com/apache/spark/pull/13861
I had to black out a few hostnames to avoid NDA issues with my clusters,
but the format should be clear.
![updated](https://cloud.githubusercontent.com/assets/1450821/16531866/4f1dca3c-3f83-11e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13980
**[Test build #61631 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61631/consoleFull)**
for PR 13980 at commit
[`02eced8`](https://github.com/apache/spark/commit/0
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14017
Thank you for review, @srowen :)
---
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
ena
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13980
Jenkins test this please
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/13980
I can't OK this, but can test it
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14017
LGTM
---
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 featur
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14016#discussion_r69341904
--- Diff: bin/pyspark ---
@@ -31,12 +31,7 @@ export _SPARK_CMD_USAGE="Usage: ./bin/pyspark [options]"
# and executor Python executables.
# Dete
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
@MLnick I like that idea! I've done the shortcut approach on the root
(since thats the shortcut we already had) - but I'll look at adding the
shortcut on the MLWriter its self as well. Thanks!
---
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69341032
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSchedu
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/13861
If you could attach new screen shots please
---
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 featu
Github user husseinhazimeh commented on the issue:
https://github.com/apache/spark/pull/13980
@srowen can you please have a look?
---
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 skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69340420
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSchedule
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69340374
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSchedu
Github user tmagrino commented on the issue:
https://github.com/apache/spark/pull/13861
Latest commit now uses the more minimalist format discussed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13704
**[Test build #61630 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61630/consoleFull)**
for PR 13704 at commit
[`256c861`](https://github.com/apache/spark/commit/2
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69339697
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSchedu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13704
**[Test build #61629 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61629/consoleFull)**
for PR 13704 at commit
[`2d3d34a`](https://github.com/apache/spark/commit/2
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13704
@cloud-fan, I checked the following
>Let me check which code portion inserts Cast in this tree.
IIUC, [this
code](https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/o
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69338825
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSchedu
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69338924
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSchedu
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14011
looks good!
---
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 i
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69338665
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -356,4 +374,233 @@ private[mesos] trait MesosSchedu
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/13950
@srowen @zsxwing @tgravescs What do you guys think of @gurvindersingh
proposal?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13981
**[Test build #61628 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61628/consoleFull)**
for PR 13981 at commit
[`04bdb3a`](https://github.com/apache/spark/commit/0
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/13984
@shivaram what do you think?
---
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 a
Github user MechCoder commented on the issue:
https://github.com/apache/spark/pull/13981
@sethah Thank you for your comments. I have addressed them. Do you have
anything else?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69337156
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -423,18 +432,35 @@ private[spark] cl
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69336408
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -423,18 +432,35 @@ private[spark] cl
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69336174
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -423,18 +432,35 @@ private[spark] cl
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r69335697
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -423,18 +432,35 @@ private[spark] cl
Github user MechCoder commented on the issue:
https://github.com/apache/spark/pull/14016
Thanks for clarifying! It might be a good time to get rid of it..
---
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 do
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13951
**[Test build #61627 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61627/consoleFull)**
for PR 13951 at commit
[`3276d41`](https://github.com/apache/spark/commit/3
Github user MechCoder commented on a diff in the pull request:
https://github.com/apache/spark/pull/13981#discussion_r69333966
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/DecisionTreeRegressorSuite.scala
---
@@ -96,6 +108,15 @@ class DecisionTreeRegressorSuite
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14017
**[Test build #61626 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61626/consoleFull)**
for PR 14017 at commit
[`1da6767`](https://github.com/apache/spark/commit/1
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/13976
no problem... sorry i had to kill it, but i wanted to get a quick jenkins
restart in. :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13976
Thank you, @shaneknapp ! :)
---
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 a
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/14017
[MINOR][BUILD] Fix Java linter errors
## What changes were proposed in this pull request?
This PR fixes the minor Java linter errors like the following.
```
-public int re
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14016
If I recall, the original motivation for preferring the `python2.7`
executable to plain `python` was that `python` might be Python 2.6 on older
CentOS / RHEL systems and certain parts of PySpark p
Github user MechCoder commented on a diff in the pull request:
https://github.com/apache/spark/pull/13981#discussion_r69332114
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/DecisionTreeRegressorSuite.scala
---
@@ -96,6 +108,15 @@ class DecisionTreeRegressorSuite
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14007
**[Test build #61625 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61625/consoleFull)**
for PR 14007 at commit
[`bd83643`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13976
**[Test build #61624 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61624/consoleFull)**
for PR 13976 at commit
[`c43a187`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14016
**[Test build #61623 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61623/consoleFull)**
for PR 14016 at commit
[`4661493`](https://github.com/apache/spark/commit/4
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/13976
jenkins, test this please
---
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 wi
Github user MechCoder commented on the issue:
https://github.com/apache/spark/pull/14016
ping @srowen @JoshRosen
---
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 wish
GitHub user MechCoder opened a pull request:
https://github.com/apache/spark/pull/14016
[SPARK-15761] [FOLLOWUP] Set DEFAULT_PYTHON to python
## What changes were proposed in this pull request?
I would like to change
```bash
if hash python2.7 2>/dev/null; then
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14007
@maropu Sure, let me resolve it. Thanks for catching it!
---
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 h
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14007#discussion_r69329991
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -215,6 +215,8 @@ abstract class Star extends LeafExp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61621/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14007
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14007
**[Test build #61621 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61621/consoleFull)**
for PR 14007 at commit
[`c84d0c5`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61620/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61620 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61620/consoleFull)**
for PR 13680 at commit
[`6a8a97f`](https://github.com/apache/spark/commit/
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14004#discussion_r69325521
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -801,6 +804,49 @@ public static UTF8String concatWs(UTF8Stri
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14004#discussion_r69325407
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ---
@@ -801,6 +804,49 @@ public static UTF8String concatWs(UTF8Stri
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13976
Thank you, @cloud-fan .
I renovated ugly testcases with `create_row()`!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13976
**[Test build #61622 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61622/consoleFull)**
for PR 13976 at commit
[`c43a187`](https://github.com/apache/spark/commit/c
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13953
See comments in the JIRA, including mine. I do not think this is in scope
for Spark or worth maintaining and troubleshooting.
---
If your project is set up for it, you can reply to this email and h
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13834
OK, I guess that's a safe enough assumption. I guess I'm worried just
because we've seen that if this doesn't happen then waitFor will hang. But I
suppose there's no reason to expect it would block a
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69323801
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/StringExpressionsSuite.scala
---
@@ -725,4 +725,43 @@ class StringExpression
Github user andreweduffy commented on the issue:
https://github.com/apache/spark/pull/13953
@petermaxlee can you give an example of how it can "get into a lot of
problems"? The use case seems weird if you're usually developing on hardware
that you control, but makes more sense if you'
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13976#discussion_r69321672
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala
---
@@ -195,3 +195,42 @@ case class Explode(child: Ex
Github user renozhang commented on the issue:
https://github.com/apache/spark/pull/13962
@vanzin I've updated codes and comments.
Please take a look, thanks a lot.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13976#discussion_r69321184
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala
---
@@ -195,3 +195,42 @@ case class Explode(child: Ex
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13967
Hi, @cloud-fan .
Could you merge this PR? :)
---
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 t
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13906
Thank you for merging, @liancheng , @cloud-fan , and @rxin .
---
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 doe
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/14007
Great work! One question;
I got weird results from this fix.
```
scala> case class DATA(a: Int)
defined class DATA
scala> val df1 = Seq(DATA(1)).toDF
df1: org.apache.spark.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14007
**[Test build #61621 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61621/consoleFull)**
for PR 14007 at commit
[`c84d0c5`](https://github.com/apache/spark/commit/c
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14007#discussion_r69319196
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -215,6 +215,8 @@ abstract class Star extends LeafExp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13962
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61619/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13962
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13962
**[Test build #61619 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61619/consoleFull)**
for PR 13962 at commit
[`d734186`](https://github.com/apache/spark/commit/
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14009
LGTM except for a minor styling issue.
---
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
en
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/14009#discussion_r69316671
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/MetadataCacheSuite.scala ---
@@ -77,12 +77,12 @@ class MetadataCacheSuite extends QueryTest with
Sh
Github user lihongliustc closed the pull request at:
https://github.com/apache/spark/pull/13925
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61620 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61620/consoleFull)**
for PR 13680 at commit
[`6a8a97f`](https://github.com/apache/spark/commit/6
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69314031
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -653,6 +657,125 @@ case class StringRPad(str:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13962
**[Test build #61619 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61619/consoleFull)**
for PR 13962 at commit
[`d734186`](https://github.com/apache/spark/commit/d
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69309179
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -653,6 +657,125 @@ case class StringRPad(st
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14007#discussion_r69307924
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -215,6 +215,8 @@ abstract class Star extends LeafExpr
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69307883
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/StringExpressionsSuite.scala
---
@@ -725,4 +725,43 @@ class StringExpression
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69307503
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -653,6 +657,125 @@ case class StringRPad(str:
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69307232
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -653,6 +657,125 @@ case class StringRPad(str:
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69307187
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -653,6 +657,125 @@ case class StringRPad(str:
Github user janplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/14008#discussion_r69307094
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -653,6 +657,125 @@ case class StringRPad(str:
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/13906
Merged to master.
@cloud-fan Sorry that I didn't notice your comment while merging it. We may
address it in follow-up ones.
---
If your project is set up for it, you can reply to this em
201 - 300 of 475 matches
Mail list logo