[GitHub] spark pull request: [spark-4691][shuffle]Code improvement for aggr...

2014-12-05 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/3553#issuecomment-65862751 Should probably just merge it into master, and thus it's independent of the 1.2 release, right? --- If your project is set up for it, you can reply to this email and ha

[GitHub] spark pull request: [spark-4691][shuffle]Code improvement for aggr...

2014-12-05 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3553#issuecomment-65862084 I would use `[SPARK-4691][Minor] Rewrite a few lines in shuffle code`. Not a big deal if you don't change this. I'll merge this once we cut the new 1.2 release. ---

[GitHub] spark pull request: [spark-4691][shuffle]Code improvement for aggr...

2014-12-05 Thread maji2014
Github user maji2014 commented on the pull request: https://github.com/apache/spark/pull/3553#issuecomment-65761712 @pwendell any idea about this title?/ --- 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 doe