[SPARK-8464][Core][Shuffle] Consider separating aggregator and non-aggregator paths in ExternalSorter#7129

Closed

ilganeli wants to merge 20 commits intoapache:masterfrom

ilganeli:SPARK-8464

Commits on Jun 30, 2015

[SPARK-8464] As a starting point, refactored ExternalSorter into two class instances, one which uses Aggregator and one which does not. Next step is to extract out common code
Ilya Ganelin
committed
Created a top level interface for ExternalSorter and created two subclasses that implement this interface to abstract away the aggregator usage.
Ilya Ganelin
committed
Compilation fixes
Ilya Ganelin
committed
Spacing
Ilya Ganelin
committed
Style fixes
Ilya Ganelin
committed
Updated creation of ExternalSorter instances to use aggregator//no-aggregator interfaces
Ilya Ganelin
committed
Style
Ilya Ganelin
committed
Added MIMA exclude
Ilya Ganelin
committed

Commits on Jul 1, 2015

Fixed error in constructor
Ilya Ganelin
committed

Commits on Jul 5, 2015

Fixing merge conflicts
Ilya Ganelin
committed

Commits on Jul 8, 2015

Fixes to MimaExcludes
Ilya Ganelin
committed
Resolved merge conflict
Ilya Ganelin
committed
style fix
Ilya Ganelin
committed
Fixing merge conflict
Ilya Ganelin
committed

Commits on Jul 9, 2015

Merge conflict
Ilya Ganelin
committed

Commits on Jul 16, 2015

Resolved merge conflicts
Ilya Ganelin
committed

Commits on Jul 23, 2015

Fixed merge conflict
Ilya Ganelin
committed

Commits on Jul 31, 2015

Moved sort updates from basicOperator.scala to sort.scala
Ilya Ganelin
committed
Import ordering and spacing/
Ilya Ganelin
committed
Spacing
Ilya Ganelin
committed