Re: Should AggregationIterator.initializeBuffer be moved down to SortBasedAggregationIterator?

2021-05-26 Thread Cheng Su
is refactored away and no longer exists. Cheng Su From: Jacek Laskowski Date: Tuesday, May 25, 2021 at 6:35 AM To: dev Subject: Should AggregationIterator.initializeBuffer be moved down to SortBasedAggregationIterator? Hi, Just found out that the only purpose

Should AggregationIterator.initializeBuffer be moved down to SortBasedAggregationIterator?

2021-05-25 Thread Jacek Laskowski
Hi, Just found out that the only purpose of AggregationIterator.initializeBuffer is to keep SortBasedAggregationIterator happy [1]. Shouldn't it be moved down to SortBasedAggregationIterator to make things clear(er)? [1] https://github.com/apache/spark/search?q=initializeBuffer Pozdrawiam,