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

DOYUNG YOON edited comment on S2GRAPH-21 at 1/15/16 5:13 AM:
-------------------------------------------------------------

Github user SteamShon commented on the pull request:

    https://github.com/apache/incubator-s2graph/pull/7#issuecomment-171873448
  
    since some PR has been merged into master after opening this PR, I merged 
master into this PR so we can just merge them after review. Please review and 
give feedbacks.



was (Author: githubbot):
Github user SteamShon commented on the pull request:

    https://github.com/apache/incubator-s2graph/pull/7#issuecomment-171873448
  
    @daewon, @hsleep , @emeth-kim, since some PR has been merged into master 
after opening this PR, I merged master into this PR so we can just merge them 
after review. Please review and give feedbacks.


> Change PostProcessBenchmarkSpec not to store and fetch test data from storage
> -----------------------------------------------------------------------------
>
>                 Key: S2GRAPH-21
>                 URL: https://issues.apache.org/jira/browse/S2GRAPH-21
>             Project: S2Graph
>          Issue Type: Test
>            Reporter: DOYUNG YOON
>            Assignee: DOYUNG YOON
>            Priority: Trivial
>              Labels: newbie, test
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> What PostProcessBenchmarkSpec supposed to do is testing performance of 
> methods on PostProcess. 
> Currently, this spec does following on setup phase. 
> # delete and re-create test Service
> # delete and re-create test Label 
> # insert test data into HBase
> also to run PostProcess`s method, it actually fetch inserted data from setup 
> phase.
> I suggest not create test Label and insert test data entirely. 
> instead we can create QueryRequest, QueryResult, QueryRequestWithResult 
> classes without actually store test data. 
> I think by using mocked edges
> # we can run this test without need for DBMS and HBase so we can concentrate 
> on PostProcess logic`s performance only. 
> # more flexible and fast test can be possible with regard to number of edges 
> in QueryResult. 
> In general, I think there should be no dependency on RDBMS + HBase for 
> benchmark specs.



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

Reply via email to