Flatten QueryPhaseResultConsumer (#116209)

No need for a separate PendingMerges nested class, this thing has a lot
of state, lets accept that (for now) and keep it simple. Having a
redundant nested class already caused a bug in #116171 and just adds
overhead both in runtime and when reading the code.

Author: original-brownbear