Github user jacques-n commented on the pull request:
https://github.com/apache/drill/pull/477#issuecomment-209714087
A quick question and an fyi. (I haven't looked at the details of the patch.)
q) Does this change the external interface of the scripts. If so, we should
probably put in the v2 branch.
fyi) In general: you should make sure that your patch is rebased as opposed
to merge.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
