[ 
https://issues.apache.org/jira/browse/TAJO-1638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14586984#comment-14586984
 ] 

ASF GitHub Bot commented on TAJO-1638:
--------------------------------------

Github user blrunner commented on the pull request:

    https://github.com/apache/tajo/pull/599#issuecomment-112221427
  
    @charsyam 
    
    Thanks for your contribution.
    But I failed to merge this patch as following
    
    ```
    Auto-merging 
tajo-core/src/test/java/org/apache/tajo/ws/rs/resources/TestQueryResultResource.java
    Auto-merging 
tajo-core/src/main/java/org/apache/tajo/ws/rs/resources/QueryResultResource.java
    CONFLICT (content): Merge conflict in 
tajo-core/src/main/java/org/apache/tajo/ws/rs/resources/QueryResultResource.java
    Automatic merge failed; fix conflicts and then commit the result.
    ```
    
    Could you check it ?


> Remove offset parameter from rest api result/{cacheId}
> ------------------------------------------------------
>
>                 Key: TAJO-1638
>                 URL: https://issues.apache.org/jira/browse/TAJO-1638
>             Project: Tajo
>          Issue Type: Improvement
>            Reporter: DaeMyung Kang
>
> in result/{cacheId} rest api.
> offset parameter can make user confused.
> because offset can't go back.
> It's only set forward(not backward)
> and add 3 new header for result response.
> X-Tajo-Offset : from start offset
> X-Tajo-Count: fetch count
> X-Tajo-EOS: if there is end stream. it sets EOS true



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to