FineAndDandy

This looks consistent, but hard to tell if it's 100% correct.

I would love if instead of deprecating this we actually made it an imperative to compact.

similar to #2300 but for integration rather than a patch

@kevinrr888 using getPluginEnv() under the covers calls getServiceEnv(), leading to the same UnsupportedOperationException. The issue is the RFileScanner uses an IteratorEnvironment implementation which implements none of these methods.

`OfflineIteratorEnvironment` has similar issues

I like this approach to managing the thrift queue. Is it reasonable to say that different processes may need a different limit?