You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by charsyam <gi...@git.apache.org> on 2015/05/28 03:56:02 UTC

[GitHub] tajo pull request: TAJO-1627 Rest API should support this type of ...

GitHub user charsyam opened a pull request:

    https://github.com/apache/tajo/pull/590

    TAJO-1627 Rest API should support this type of query "select 1"

    Currently, Tajo Rest API supports well "select * from table" or "select count from table" <-- normal type
    but, Tajo Rest API couldn't support this type of query like "select 1" <-- direct return type
    to solve this, queries api should support resultset also.
    so I suggest this.
    queries return 201 Created when it is normal type of query
    and query is direct query type and return 200 with result
    like this (direct return type)
    {"resultCode":"OK","schema":{"fields":[{"name":"?number","typeDesc":{"dataType":
    {"type":"INT4"}
    }}],"fieldsByQualifiedName":
    {"?number":0}
    ,"fieldsByName":{"?number":[0]}},"serializedTupes":["AAAAAAE\u003d"]}

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

    $ git pull https://github.com/charsyam/tajo feature/query-with-direct-return

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

    https://github.com/apache/tajo/pull/590.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 #590
    
----
commit cd84bbe57a04a5a52e49768a232d104d42d79c56
Author: clark.kang <cl...@kakao.com>
Date:   2015-05-28T01:54:07Z

    TAJO-1627

----


---
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] tajo pull request: TAJO-1627 Rest API should support this type of ...

Posted by hyunsik <gi...@git.apache.org>.
Github user hyunsik commented on the pull request:

    https://github.com/apache/tajo/pull/590#issuecomment-108683006
  
    Hi @charsyam,
    
    The proposal looks good to me. I have one suggestion. The JSON schema for Tajo schema looks different from those of other APIs. It would be great if they are consistent. 


---
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.
---