Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9142#issuecomment-149799167
@zjffdu can you reopen this?
---
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 lepfhty commented on a diff in the pull request:
https://github.com/apache/spark/pull/9137#discussion_r42589236
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/JdbcDialects.scala ---
@@ -207,6 +225,23 @@ case object PostgresDialect extends JdbcDialect {
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149799071
Thanks - I've merged this.
---
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 th
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149798452
@rxin, I believe that the doctest itself reflects the before/after.
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9169#issuecomment-149798260
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9169#issuecomment-149798259
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9169#issuecomment-149798156
**[Test build #44049 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44049/consoleFull)**
for PR 9169 at commit
[`1888342`](https://git
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149797785
Can you show the before and after changes?
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149795220
**[Test build #44050 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44050/consoleFull)**
for PR 9193 at commit
[`0504092`](https://gith
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9193#discussion_r42587435
--- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala
---
@@ -224,7 +224,8 @@ private[r] class RBackendHandler(server: RBackend)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-149793990
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-149793988
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-149793919
**[Test build #44047 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44047/consoleFull)**
for PR 9192 at commit
[`b0348d7`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149793886
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149793899
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149792826
**[Test build #44048 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44048/consoleFull)**
for PR 9181 at commit
[`f5e7c4f`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9169#issuecomment-149792624
**[Test build #44049 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44049/consoleFull)**
for PR 9169 at commit
[`1888342`](https://gith
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/9179#issuecomment-149792487
@shivaram, I will update the documentation
---
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 projec
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42586670
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/joinTypes.scala
---
@@ -47,4 +48,9 @@ case object RightOuter extends JoinTyp
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9169#issuecomment-149792274
Merged build started.
---
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/9169#issuecomment-149792262
Merged build triggered.
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/9169#issuecomment-149791995
All done, thank you. Yes I think there's a good reason MLlib has more
`@Experimental`, also simply because it makes use of this feature a lot. ML
pipelines is still mostl
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-149791993
**[Test build #44047 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44047/consoleFull)**
for PR 9192 at commit
[`b0348d7`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149791758
Merged build started.
---
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/9181#issuecomment-149791747
Merged build triggered.
---
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 cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/7417#issuecomment-149791623
Hi @Sephiroth-Lin , according to the previous discussion, I think we should
NOT do optimization according to data size, do you mind closing this PR and
help us review
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42586165
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -408,6 +552,25 @@ class Analyzer(
}
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149791460
> @zsxwing I think this duplicated the existing issue / PR #9096 . Does
this cover the same logic, and, can you include all of the changes here?
This is a differ
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-149791268
Merged build started.
---
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/9192#issuecomment-149791254
Merged build triggered.
---
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 zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149791252
retest 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 fea
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-149791103
Rebase to master. Update to fix the new `as.DataFrame` method.
@davies Thanks! Appreciated. I'd like to leave R doc update separated if
that's ok by you - to
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/9123#issuecomment-149790801
ping @yhuai
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149790611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149790609
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149790533
**[Test build #44044 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44044/consoleFull)**
for PR 9177 at commit
[`39f3fc0`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149790455
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149790428
**[Test build #44046 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44046/consoleFull)**
for PR 9003 at commit
[`62c515c`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149790453
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
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42585365
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -270,6 +273,146 @@ class Analyzer(
}
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-149790122
@felixcheung This looks great overall, could you also update all the
examples?
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149790070
@zsxwing I think this duplicated the existing issue / PR
https://github.com/apache/spark/pull/9096 . Does this cover the same logic,
and, can you include all of the chang
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/9096#issuecomment-149790022
It looks like this is a duplicate/subset of
https://github.com/apache/spark/pull/9181 -- is that covering the same ground?
---
If your project is set up for it, you can
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42584878
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -857,3 +857,329 @@ object HyperLogLogPlusPlus
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/9145#issuecomment-149788964
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
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42584819
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -857,3 +857,329 @@ object HyperLogLogPlusPlus
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9145#discussion_r42584730
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/columnar/GenerateColumnAccessor.scala
---
@@ -0,0 +1,149 @@
+/*
+ * Licensed to the Apache S
Github user navis commented on the pull request:
https://github.com/apache/spark/pull/8899#issuecomment-149788617
Then, the problem would be which one is better. #9142 looks more intuitive
and this provides a chance to handle the situation(invalid path) for input
formatter.
---
If y
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149788655
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149788654
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
Github user sethah commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149788583
@mengxr Addressed all comments and rearranged the class inheritances for
case classes. I made another abstract class `SecondMoment` which all `Variance`
classes inherit f
Github user marsishandsome commented on the pull request:
https://github.com/apache/spark/pull/9168#issuecomment-149788293
Hi all,
I have updated the patch and only use Hadoop's public stable API.
I will submit a patch to Hadoop.
This patch is just a workaround and will
Github user harishreedharan commented on the pull request:
https://github.com/apache/spark/pull/8942#issuecomment-149787894
If the current user's ugi is what is used by the FileSystem cache, this
should not really be an issue no? Because we actually do update the current
user's creden
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149787874
**[Test build #44046 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44046/consoleFull)**
for PR 9003 at commit
[`62c515c`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149787825
**[Test build #44044 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44044/consoleFull)**
for PR 9177 at commit
[`39f3fc0`](https://gith
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42584417
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -842,3 +699,302 @@ object HyperLogLogPlusPlus
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42584440
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -842,3 +699,302 @@ object HyperLogLogPlusPlus
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42584447
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -842,3 +699,302 @@ object HyperLogLogPlusPlus
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149787044
Merged build started.
---
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/9003#issuecomment-149786976
Merged build triggered.
---
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 sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42584406
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -842,3 +699,302 @@ object HyperLogLogPlusPlus
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149785661
Merged build started.
---
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/9003#issuecomment-149785650
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9177#issuecomment-149785645
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149785662
Merged build started.
---
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8026#issuecomment-149785524
@chenghao-intel, thanks a bunch for updating this; the current version of
this patch is a lot easier to understand and I'm happy with how clean the code
turned out. I
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42584123
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -270,6 +273,146 @@ class Analyzer(
}
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42584080
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/sources/hadoopFsRelationSuites.scala
---
@@ -510,21 +510,39 @@ abstract class HadoopFsRelationTest e
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42584058
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42584044
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42584024
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42583983
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42583969
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42583944
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149784768
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149784764
**[Test build #44043 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44043/consoleFull)**
for PR 9193 at commit
[`ed57eb8`](https://git
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42583933
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -270,6 +273,146 @@ class Analyzer(
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149784742
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149784767
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149784692
**[Test build #44035 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44035/consoleFull)**
for PR 9181 at commit
[`f5e7c4f`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9181#issuecomment-149784741
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
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42583857
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149784461
**[Test build #44043 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44043/consoleFull)**
for PR 9193 at commit
[`ed57eb8`](https://gith
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42583623
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -270,6 +273,146 @@ class Analyzer(
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9193#issuecomment-149783538
Merged build started.
---
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/9193#issuecomment-149783525
Merged build triggered.
---
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 chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42583551
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -270,6 +273,146 @@ class Analyzer(
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149783443
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149783421
**[Test build #44039 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44039/consoleFull)**
for PR 9003 at commit
[`4ae234e`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-149783441
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8026#issuecomment-149783377
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8026#issuecomment-149783376
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
GitHub user sun-rui opened a pull request:
https://github.com/apache/spark/pull/9193
[SPARK-11209][SPARKR] Add window functions into SparkR [step 1].
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sun-rui/spark SPARK-11209
Alte
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8026#issuecomment-149783351
**[Test build #44040 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44040/consoleFull)**
for PR 8026 at commit
[`9f08f76`](https://git
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42583480
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -270,6 +273,146 @@ class Analyzer(
}
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42583439
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -270,6 +273,146 @@ class Analyzer(
}
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42583413
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -69,6 +71,7 @@ class Analyzer(
WindowsSu
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42583415
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8026#discussion_r42583431
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -544,11 +544,35 @@ abstract class HadoopFsRelation
private[sql](mayb
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/9055#discussion_r42583339
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -1485,14 +1490,39 @@
https://cwiki.apache.org/confluence/display/Hive/E
1 - 100 of 912 matches
Mail list logo