Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6780
---
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 enab
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-115001504
Thanks! Merged to master.
---
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/6780#issuecomment-114788600
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/6780#issuecomment-114788562
[Test build #35652 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35652/console)
for PR 6780 at commit
[`34aa07b`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114748738
[Test build #35652 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35652/consoleFull)
for PR 6780 at commit
[`34aa07b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114747598
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/6780#issuecomment-114747637
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 cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114747461
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114745263
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/6780#issuecomment-114745188
[Test build #35634 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35634/console)
for PR 6780 at commit
[`34aa07b`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114724791
[Test build #35634 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35634/consoleFull)
for PR 6780 at commit
[`34aa07b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114724550
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/6780#issuecomment-114724542
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/6780#issuecomment-114587583
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/6780#issuecomment-114587448
[Test build #35558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35558/console)
for PR 6780 at commit
[`72c3f69`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114562787
[Test build #35558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35558/consoleFull)
for PR 6780 at commit
[`72c3f69`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-114562054
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/6780#issuecomment-114562018
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 marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6780#discussion_r32991598
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -106,12 +103,16 @@ case class InsertIntoHiveTable(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6780#discussion_r32902049
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -106,12 +103,16 @@ case class InsertIntoHiveTable(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113949998
[Test build #35410 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35410/console)
for PR 6780 at commit
[`b65dd6b`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113950001
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/6780#issuecomment-113944947
[Test build #35410 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35410/consoleFull)
for PR 6780 at commit
[`b65dd6b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113944426
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/6780#issuecomment-113944400
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/6780#issuecomment-113943127
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/6780#issuecomment-113943049
[Test build #35408 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35408/console)
for PR 6780 at commit
[`e5c19c8`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113941696
[Test build #35406 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35406/console)
for PR 6780 at commit
[`9420510`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113941706
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/6780#issuecomment-113938328
[Test build #35408 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35408/consoleFull)
for PR 6780 at commit
[`e5c19c8`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113938201
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/6780#issuecomment-113938207
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/6780#issuecomment-113937702
[Test build #35406 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35406/consoleFull)
for PR 6780 at commit
[`9420510`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113937693
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/6780#issuecomment-113937687
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/6780#issuecomment-113937616
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-11397
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/6780#issuecomment-113933322
[Test build #35400 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35400/console)
for PR 6780 at commit
[`9420510`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113926937
[Test build #35400 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35400/consoleFull)
for PR 6780 at commit
[`9420510`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113926409
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/6780#issuecomment-113926383
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113918454
[Test build #35395 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35395/console)
for PR 6780 at commit
[`ec57bad`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113918463
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/6780#issuecomment-113911003
[Test build #35395 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35395/consoleFull)
for PR 6780 at commit
[`ec57bad`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113910814
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/6780#issuecomment-113910811
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/6780#issuecomment-113880280
cc @marmbrus , `NotSerializableException` error will exist at
`HiveCompatibilitySuite` when insert into hive table.
---
If your project is set up for it, you can repl
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113395075
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/6780#issuecomment-113395050
[Test build #35230 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35230/console)
for PR 6780 at commit
[`d11fa9e`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113382396
[Test build #35230 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35230/consoleFull)
for PR 6780 at commit
[`d11fa9e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-113382300
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/6780#issuecomment-113382314
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 marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/6780#discussion_r32668112
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -160,17 +166,22 @@ case class TakeOrdered(limit: Int, sortOrder:
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-112834131
Hmm...this PR doesn't change the handling of `Limit(Sort(Aggregate))`, it
just swap `Limit` and `Project`(and combine them) for case
`Project(Limit(Sort(...)))`, is it
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-112664289
How about the `Limit(Sort(Aggregate))`, will that be a bug? Sorry, I didn't
test it, can you confirm that? Probably it's not a good idea to put 3
functionalities
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6780#discussion_r32394261
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -160,17 +166,22 @@ case class TakeOrdered(limit: Int, sortOrder
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111932724
[Test build #34904 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34904/console)
for PR 6780 at commit
[`0d53727`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111932787
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/6780#issuecomment-111912520
[Test build #34904 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34904/consoleFull)
for PR 6780 at commit
[`0d53727`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111912002
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/6780#issuecomment-111911994
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/6780#issuecomment-111910688
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/6780#issuecomment-111910677
[Test build #34901 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34901/console)
for PR 6780 at commit
[`edfdc6f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111905596
[Test build #34901 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34901/consoleFull)
for PR 6780 at commit
[`edfdc6f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111905415
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/6780#issuecomment-111905428
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/6780#issuecomment-111856384
[Test build #34892 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34892/console)
for PR 6780 at commit
[`91d798d`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111856389
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/6780#issuecomment-111854594
[Test build #34892 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34892/consoleFull)
for PR 6780 at commit
[`91d798d`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111854468
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/6780#issuecomment-111854434
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111852504
[Test build #34890 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34890/console)
for PR 6780 at commit
[`3c7dab0`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111852511
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/6780#issuecomment-111851845
[Test build #34890 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34890/consoleFull)
for PR 6780 at commit
[`3c7dab0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6780#issuecomment-111851807
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/6780#issuecomment-111851812
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
76 matches
Mail list logo
