Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-214131211
Close this for now and, if needed, I'll 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 y
Github user maropu closed the pull request at:
https://github.com/apache/spark/pull/10631
---
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 ena
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-173866941
@rxin Okay and I'll wait for that.
---
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 n
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-173787354
I will add this to the review queue. But to be honest this API is super
hard to design and I am not sure if there is an easy solution. The priority is
also not very high s
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-173552264
@rxin ping
---
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 maropu commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171866035
@rxin Could you give me any suggestion on this workaround?
https://github.com/apache/spark/pull/10631/files#diff-d99813bd5bbc18277e4090475e4944cfR130
---
If your pr
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171723257
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171723261
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171722910
**[Test build #49402 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49402/consoleFull)**
for PR 10631 at commit
[`736a3ab`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171690890
**[Test build #49402 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49402/consoleFull)**
for PR 10631 at commit
[`736a3ab`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171681629
**[Test build #49401 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49401/consoleFull)**
for PR 10631 at commit
[`9f039e6`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171681713
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171681714
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171677678
**[Test build #49401 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49401/consoleFull)**
for PR 10631 at commit
[`9f039e6`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171659734
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171659736
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171659679
**[Test build #49400 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49400/consoleFull)**
for PR 10631 at commit
[`5f6441d`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171656357
**[Test build #49400 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49400/consoleFull)**
for PR 10631 at commit
[`5f6441d`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171637085
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171637086
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171637079
**[Test build #49399 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49399/consoleFull)**
for PR 10631 at commit
[`c9cb213`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171635825
**[Test build #49399 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49399/consoleFull)**
for PR 10631 at commit
[`c9cb213`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171635572
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-171635576
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-169619635
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-169619162
**[Test build #48922 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48922/consoleFull)**
for PR 10631 at commit
[`42bc664`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-169593029
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 projec
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10631#discussion_r49049257
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -301,6 +301,8 @@ trait PrunedScan {
@DeveloperApi
trait PrunedF
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-169590863
**[Test build #48920 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48920/consoleFull)**
for PR 10631 at commit
[`aa1607c`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-169590871
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/10631#discussion_r49049015
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -286,7 +286,7 @@ trait PrunedScan {
*/
@DeveloperApi
tr
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10631#issuecomment-169565599
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10631#discussion_r49044190
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/sources/interfaces.scala ---
@@ -286,7 +286,7 @@ trait PrunedScan {
*/
@DeveloperApi
trai
GitHub user maropu opened a pull request:
https://github.com/apache/spark/pull/10631
[SPARK-12686][SQL] Support group-by push down to data sources
This pr enables pushed-down MIN/MAX aggregation into JDBC data sources. As
for logical plan nodes like 'Aggregate -> Project -> (Filter)
34 matches
Mail list logo