I agree this looks scary and it caused a bug or two, but this a proven the give a good performance improvement when a low amount of docs match. Otherwise there will be a lot of unnecessary get calls to the bitset (lets say 50 docs match, but there are 3M parent docs in total) and the overhead of the shortcircuit mechanism is low.