[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-08-04 Thread ggupta81
Github user ggupta81 commented on the pull request: https://github.com/apache/spark/pull/7233#issuecomment-127815741 Closing this patch since we have #7928 open. --- 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 pro

[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-08-04 Thread ggupta81
Github user ggupta81 closed the pull request at: https://github.com/apache/spark/pull/7233 --- 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 en

[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-08-04 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7233#issuecomment-127692251 @ggupta81 can you close this patch then since we already have #7928? It's confusing to have both open. --- If your project is set up for it, you can reply to this em

[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-08-04 Thread ggupta81
Github user ggupta81 commented on the pull request: https://github.com/apache/spark/pull/7233#issuecomment-127545776 I have opened a JIRA issue: https://issues.apache.org/jira/browse/SPARK-9592 and created pull request on master: https://github.com/apache/spark/pull/792

[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-08-01 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7233#issuecomment-126943691 Actually, @rxin or @yhuai, is this a bug that should be fixed for 1.5? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-08-01 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7233#issuecomment-126943661 Let's close this issue for now. --- 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] spark pull request: Fixed First and Last aggregates to compute fir...

2015-07-06 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7233#issuecomment-118958414 @ggupta81 please file a JIRA and open a patch against the master branch. For more detail see https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark a

[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-07-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7233#issuecomment-118727714 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 pr

[GitHub] spark pull request: Fixed First and Last aggregates to compute fir...

2015-07-05 Thread ggupta81
GitHub user ggupta81 opened a pull request: https://github.com/apache/spark/pull/7233 Fixed First and Last aggregates to compute first and last value per G… In current implementation, First and Last aggregates were calculating the values for entire DataFrame partition and then the