Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13067#discussion_r64172019
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -199,20 +200,123 @@ class AstBuilder extends
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13067#discussion_r64171963
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala ---
@@ -50,6 +50,10 @@ private[hive] trait HiveStrategies {
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13067#discussion_r64171948
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -374,6 +375,17 @@ case class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13067#discussion_r64171821
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -374,6 +375,17 @@ case class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13067#discussion_r64171525
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -198,6 +198,7 @@ query
insertInto
: INSERT
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13067#discussion_r64171510
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -713,6 +714,7 @@ WITH: 'WITH';
VALUES: 'VALUES';
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13243#issuecomment-220890649
**[Test build #59122 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59122/consoleFull)**
for PR 13243 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9759#issuecomment-220889923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9759#issuecomment-220889921
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9759#issuecomment-220889842
**[Test build #59119 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59119/consoleFull)**
for PR 9759 at commit
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/8559#discussion_r64170026
--- Diff:
core/src/test/resources/META-INF/services/org.apache.spark.scheduler.ExternalClusterManager
---
@@ -1 +1,2 @@
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13209#issuecomment-220887533
**[Test build #59121 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59121/consoleFull)**
for PR 13209 at commit
Github user markhamstra commented on a diff in the pull request:
https://github.com/apache/spark/pull/8559#discussion_r64169565
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskResult.scala
---
@@ -59,7 +59,7 @@ private[spark] class DirectTaskResult[T](
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13209#issuecomment-220887426
**[Test build #3010 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3010/consoleFull)**
for PR 13209 at commit
Github user sameeragarwal commented on the pull request:
https://github.com/apache/spark/pull/13209#issuecomment-220887288
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13209#issuecomment-220886852
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13209#issuecomment-220886854
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13209#issuecomment-220886847
**[Test build #59120 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59120/consoleFull)**
for PR 13209 at commit
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13257#issuecomment-220886604
Hi @sbcd90 I am not a committer but I just left a comment because I like
your PR. Let me add some more comments which I think might be changed.
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13209#issuecomment-220886265
**[Test build #59120 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59120/consoleFull)**
for PR 13209 at commit
Github user sbcd90 commented on the pull request:
https://github.com/apache/spark/pull/13257#issuecomment-220885344
Hello @HyukjinKwon , I have added a testcase now. Please review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13243#issuecomment-220883772
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13243#issuecomment-220883773
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13243#issuecomment-220883664
**[Test build #59118 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59118/consoleFull)**
for PR 13243 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64166173
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,54 @@ case class NewInstance(
Github user wangyang1992 commented on the pull request:
https://github.com/apache/spark/pull/13177#issuecomment-220880826
Hi @andrewor14, sorry to bother you, but does this pr need to be further
refined, or it is ready to merge? Could you please give me some instructions?
Thanks.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9759#issuecomment-220880780
**[Test build #59119 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59119/consoleFull)**
for PR 9759 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/9759#discussion_r64166002
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -129,13 +129,14 @@ object JacksonParser extends
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13245#issuecomment-220880599
Sorry, it wasn't a good idea of extending SparkSession for RDD. I'm closing
this PR.
---
If your project is set up for it, you can reply to this email and have
Github user dongjoon-hyun closed the pull request at:
https://github.com/apache/spark/pull/13245
---
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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9759#discussion_r64164980
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -129,13 +129,14 @@ object JacksonParser extends
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/9759#discussion_r64164602
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -129,13 +129,14 @@ object JacksonParser extends
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64164066
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,55 @@ case class NewInstance(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13243#issuecomment-220876025
**[Test build #59118 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59118/consoleFull)**
for PR 13243 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64163700
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,55 @@ case class NewInstance(
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/13169#issuecomment-220875681
thanks, merging to master and 2.0!
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13169
---
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 the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220875134
LGTM, cc @yhuai to take another 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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64162830
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,55 @@ case class NewInstance(
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64162262
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,55 @@ case class NewInstance(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13254#issuecomment-220870837
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13254#issuecomment-220870836
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13254#issuecomment-220870762
**[Test build #59117 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59117/consoleFull)**
for PR 13254 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/13116#discussion_r64160441
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -51,6 +49,7 @@ import
org.apache.spark.sql.execution.python.EvaluatePython
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13254#issuecomment-220866525
**[Test build #59117 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59117/consoleFull)**
for PR 13254 at commit
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13254#issuecomment-220866532
@andrewor14 Could you please take a look maybe?
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220864269
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220864230
**[Test build #59116 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59116/consoleFull)**
for PR 13093 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220864268
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12850#issuecomment-220862945
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12850#issuecomment-220862943
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13236#issuecomment-220862905
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13236#issuecomment-220862906
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12850#issuecomment-220862881
**[Test build #59114 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59114/consoleFull)**
for PR 12850 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13236#issuecomment-220862857
**[Test build #59115 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59115/consoleFull)**
for PR 13236 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220860884
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220860885
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220860842
**[Test build #59113 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59113/consoleFull)**
for PR 13087 at commit
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13236#discussion_r64157490
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -73,7 +74,7 @@ import org.apache.spark.util.{CircularBuffer,
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/13236#discussion_r64157471
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -73,7 +74,7 @@ import org.apache.spark.util.{CircularBuffer,
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13257#issuecomment-220860596
@sbcd90 I think we will need a test to make sure this fixes the issue and
other changes in the future do not break this change.
---
If your project is set up for
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13257#issuecomment-220860424
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
GitHub user sbcd90 opened a pull request:
https://github.com/apache/spark/pull/13257
[SPARK-15474][SQL]ORC data source fails to write and read back empty
dataframe
## What changes were proposed in this pull request?
Currently ORC data source fails to write and read empty
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220860196
**[Test build #59116 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59116/consoleFull)**
for PR 13093 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13236#issuecomment-220859959
**[Test build #59115 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59115/consoleFull)**
for PR 13236 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12850#issuecomment-220859004
**[Test build #59114 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59114/consoleFull)**
for PR 12850 at commit
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220858268
Thank you, @felixcheung ! Then, this PR is enough for the current master
branch. :)
If SparkR have something related to this in the future, we can add a note
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220858022
@dongjoon-hyun I don't think we support UDF this way in SparkR?
Maybe something we could add to `dapply`? @sun-rui
---
If your project is set up for it, you
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13093#discussion_r64156688
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -433,6 +433,18 @@ class SessionCatalog(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13093#discussion_r64156652
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -441,7 +441,15 @@ class Analyzer(
object
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/13093#discussion_r64156604
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -441,7 +441,15 @@ class Analyzer(
object
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/13116#issuecomment-220857549
@burness that is no problem. I have taken a look at the current
implementation of the `RDD`'s `takeSample`; it is very similar to your
implementation. I do still
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220857498
I added for Scaladoc/Javadoc/Pydoc, but I cannot find proper places exposed
in SparkR doc.
Hi, @shivaram , @davies , @felixcheung .
Should we need to
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220857000
**[Test build #59113 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59113/consoleFull)**
for PR 13087 at commit
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220855166
All issues are resolved now. I'll update this PR now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13087#issuecomment-220855050
Thank you for sharing this, @linbojin .
---
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
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13192#issuecomment-220851707
Hi, @davies and @rxin .
Could you review this PR again when you have some time?
---
If your project is set up for it, you can reply to this email and have
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13239#discussion_r64154982
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -359,8 +359,8 @@ private[sql] abstract class
Github user mbaddar1 commented on the pull request:
https://github.com/apache/spark/pull/11729#issuecomment-220850123
@srowen @jkbradley Will work on the comments added by @jkbradley and will
update the PR ASAP
---
If your project is set up for it, you can reply to this email and
Github user tejasapatil commented on the pull request:
https://github.com/apache/spark/pull/12194#issuecomment-220848477
Can anyone please review 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220847276
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220847275
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13212#issuecomment-220847232
**[Test build #59111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59111/consoleFull)**
for PR 13212 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13169#issuecomment-220846483
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13169#issuecomment-220846484
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13169#issuecomment-220846427
**[Test build #59112 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59112/consoleFull)**
for PR 13169 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9759#discussion_r64153280
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala
---
@@ -129,13 +129,14 @@ object JacksonParser
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/13243#issuecomment-220844617
mostly LGTM except some minor comments, thanks for working on it!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64153183
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,55 @@ case class NewInstance(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64153170
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,55 @@ case class NewInstance(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13243#discussion_r64153124
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -232,27 +232,55 @@ case class NewInstance(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13093#discussion_r64153082
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -433,6 +433,18 @@ class SessionCatalog(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/13093#discussion_r64153076
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -441,7 +441,15 @@ class Analyzer(
object
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/13256#issuecomment-220843098
@Luwein close 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 this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13256#issuecomment-220842900
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
GitHub user Luwein opened a pull request:
https://github.com/apache/spark/pull/13256
Master
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/spark master
Alternatively you can review and apply these changes as the patch
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220842321
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220842319
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13093#issuecomment-220842261
**[Test build #59110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59110/consoleFull)**
for PR 13093 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13169#issuecomment-220842129
**[Test build #59112 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59112/consoleFull)**
for PR 13169 at commit
1 - 100 of 253 matches
Mail list logo
