GitHub user ilganeli opened a pull request:

    https://github.com/apache/spark/pull/3666

    [SPARK-4569] Rename 'externalSorting' in Aggregator

    Hi all - I've renamed the unhelpfully named variable and added a comment 
clarifying what's actually happening. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ilganeli/spark SPARK-4569B

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/3666.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3666
    
----
commit 5b3f39cf4f1475a4b656eb24d563af80e4a953c9
Author: Ilya Ganelin <ilya.gane...@capitalone.com>
Date:   2014-12-10T17:51:42Z

    [SPARK-4569] Rename  in Aggregator

commit d7cefec06e0e3b235ee67bcdf8bf115c92a1cbed
Author: Ilya Ganelin <ilya.gane...@capitalone.com>
Date:   2014-12-10T17:52:40Z

    [SPARK-4569] Rename 'externalSorting' in Aggregator

commit e2d20929b043ed4dbe1001bb38e3e441c8450992
Author: Ilya Ganelin <ilya.gane...@capitalone.com>
Date:   2014-12-10T17:53:53Z

    [SPARK-4569] Rename 'externalSorting' in Aggregator

commit 18103943e4b2584ce3079f466cdd7e3253675fac
Author: Ilya Ganelin <ilya.gane...@capitalone.com>
Date:   2014-12-10T17:54:39Z

    [SPARK-4569] Rename 'externalSorting' in Aggregator

----


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to