GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/8973
[WIP] Remove unused initialInputBufferOffset argument from TungstenAggregate
(Will open a JIRA if / when we determine that this minor refactoring is
valid).
This patch removes an unused `initialInputBufferOffset` field from
`TungstenAggregate` and `TungstenAggregationIterator`.
/cc @yhuai, does the fact that this field was unused indicate some sort of
bug? It seems that we can't remove it from the sort-based aggregate.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JoshRosen/spark tungsten-agg-comments
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/8973.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 #8973
----
commit 37a0731dc13da179cf701d6a4c5cd56ed1db7615
Author: Josh Rosen <[email protected]>
Date: 2015-10-04T01:51:08Z
Remove unused initialInputBufferOffset from TungstenAggregate.
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
