Github user jacques-n commented on the pull request:
https://github.com/apache/drill/pull/374#issuecomment-184037187
Looks reasonable. I added a few comments.
I think the Mongo interface => class changes should be a separate patch. It
adds noise to this patch and is really a completely unrelated issue. I also
think it touches a lot of files and wonder whether it should ordered in a
special way versus other outstanding mongo patches.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
