Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/978
@sohami, pushed a commit that addresses your PR comments.
Regarding the "one stop shop" comment. This PR removes the
`OperExecContextImpl` class. That class was an earlier attempt to combine the
three items into one. Additional experience showed that the existing operator
context could do that task instead.
This PR did not change existing operators to avoid passing around multiple
items. Instead, it allows new code (for the batch size limitation project) to
pass just the operator context, and use that to obtain the other two items. The
external sort had to be changed because it used the old `OperExecContextImpl`
for that purpose, and so the code had to be updated when `OperExecContextImpl`
was removed.
---
