I think this would be better off in ```buildExecutor(...)```, particularly since it is bloated with several lines of comments. It breaks up the basic scanning logic, making it more difficult to follow, and it really a configuration detail of the executor that I don't care about at this level of abstraction.