Github user jihoonson commented on the pull request:
https://github.com/apache/tajo/pull/13#issuecomment-46403452
Hyunsik, thanks for your contribution.
I tried to review the patch carefully, and left some comments.
In addition, I have one more question.
In your patch, bound types of window frame are divided into two groups
according to whether each of them is the start bound or end bound. Thus, an
window frame has one of unbounded preceding, value preceding, or current row as
its start bound and one of unbounded following, value following, or current row
as its end bound.
But I found that precedings(followings) are also allowed as start(end)
bound in PostgreSQL.
(Here is the link to bound restrictions in PostgreSQL.
http://www.postgresql.org/docs/9.1/static/sql-expressions.html#SYNTAX-WINDOW-FUNCTIONS)
Do you have any reasons of this restriction?
---
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.
---
