Github user amansinha100 commented on the pull request:
https://github.com/apache/drill/pull/322#issuecomment-172038682
Overall refactoring looks ok. One thing that needs some clarity: Is the
definition of default frame 'between unbounded preceding and current row' ? If
the user explicitly specifies this frame in the query, will it be treated as
default or custom frame (as opposed to not specifying anything frame) ?
---
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.
---
