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

ASF GitHub Bot commented on S2GRAPH-9:
--------------------------------------

Github user SteamShon commented on the pull request:

    https://github.com/apache/incubator-s2graph/pull/8#issuecomment-171879519
  
    @daewon  so it seems that this PR is resolving S2GRAPH-9 and S2GRAPH-27 at 
the same time. I think it is better for us to make issue and related PR small 
to make reviewer`s life easier. But I guess this time, it seems reasonable to 
resolve two issue at the same time. please close S2GRAPH-27 also when you close 
this issue. PR itself looks good to me. +1


> Provide rest server using netty
> -------------------------------
>
>                 Key: S2GRAPH-9
>                 URL: https://issues.apache.org/jira/browse/S2GRAPH-9
>             Project: S2Graph
>          Issue Type: Sub-task
>            Reporter: DOYUNG YOON
>            Assignee: Daewon Jeong
>            Priority: Minor
>              Labels: project_structure, refactoring
>   Original Estimate: 240h
>  Remaining Estimate: 240h
>
> I benchmarked with netty and it has much better performance with much less 
> resources on query. since faster query means a lot, I think it would be 
> better to add query rest server base on netty. also by implementing query 
> server on different framework, we can find which codes are common and which 
> are server framework specific.



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

Reply via email to