You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@eagle.apache.org by wangyum <gi...@git.apache.org> on 2016/06/08 16:02:43 UTC

[GitHub] incubator-eagle pull request #230: EAGLE-333 Fix hive.ql.Parser can't parse ...

GitHub user wangyum opened a pull request:

    https://github.com/apache/incubator-eagle/pull/230

    EAGLE-333 Fix hive.ql.Parser can't parse complex SQL.

    1. Use parseTokFunction() to parse HiveParser.TOK_FUNCTIONDI type.
    
    [EAGLE-333](https://issues.apache.org/jira/browse/EAGLE-333)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/wangyum/incubator-eagle EAGLE-333

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-eagle/pull/230.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #230
    
----
commit 4d58b4e45f90607816fe9bf7e0bf4438f622ff6c
Author: Yuming Wang <wg...@gmail.com>
Date:   2016-06-08T15:49:40Z

    Fix hive.ql.Parser can't parse complex SQL.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle issue #230: EAGLE-333 Fix hive.ql.Parser can't parse complex...

Posted by pkuwm <gi...@git.apache.org>.

Github user pkuwm commented on the issue:

    https://github.com/apache/incubator-eagle/pull/230
  
    Reviewed.
    This fix is for "distinct" which we did not support before. So add "TOK_FUNCTIONDI" in the parser to parse "distinct".
    Looks good to me. It can be merged.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request #230: EAGLE-333 Fix hive.ql.Parser can't parse ...

Posted by wangyum <gi...@git.apache.org>.

Github user wangyum closed the pull request at:

    https://github.com/apache/incubator-eagle/pull/230


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle issue #230: EAGLE-333 Fix hive.ql.Parser can't parse complex...

Posted by yonzhang <gi...@git.apache.org>.

Github user yonzhang commented on the issue:

    https://github.com/apache/incubator-eagle/pull/230
  
    Please close this PR. This is merged into branch 0.4 https://github.com/apache/incubator-eagle/commit/11d4be769b9cc1decd97f9d18f96236afbf153bf


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---