[jira] [Updated] (CALCITE-2263) Wrong start Pos in SqlOrderBy

["Wenlong Lyu (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CALCITE-2263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Wenlong Lyu updated CALCITE-2263:
---------------------------------
    Description: 
currently, start pos of SqlOrderBy is pos of limit, by I think it should be the start pos of the ordered query.
{code:java}
// Parser.jj
if (orderBy != null || start != null || count != null) {
if (orderBy == null) {
orderBy = SqlNodeList.EMPTY;
}
e = new SqlOrderBy(getPos(), e, orderBy, start, count);

}
return e;{code}

  was:
{code:java}
// Parser.jj
if (orderBy != null || start != null || count != null) {
if (orderBy == null) {
orderBy = SqlNodeList.EMPTY;
}
e = new SqlOrderBy(getPos(), e, orderBy, start, count);

}
return e;{code}


> Wrong start Pos in SqlOrderBy
> -----------------------------
>
>                 Key: CALCITE-2263
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2263
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Wenlong Lyu
>            Assignee: Julian Hyde
>            Priority: Major
>
> currently, start pos of SqlOrderBy is pos of limit, by I think it should be the start pos of the ordered query.
> {code:java}
> // Parser.jj
> if (orderBy != null || start != null || count != null) {
> if (orderBy == null) {
> orderBy = SqlNodeList.EMPTY;
> }
> e = new SqlOrderBy(getPos(), e, orderBy, start, count);
> }
> return e;{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)