It would be nice if we could get rid of this m_outputTable data member too.
The abstract executor defines "TempTable* m_tmpOutputTable;" for about the same purpose but it does strange things with that member that I don't completely trust -- sometimes it clears the table after init (why?).
Yet, that m_tmpOutputTable member seems to work fine for aggregateexecutor, so I guess it's OK to use?