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

jirapos...@reviews.apache.org commented on HBASE-5717:
------------------------------------------------------



bq.  On 2012-04-05 19:12:39, Lars Hofhansl wrote:
bq.  > /src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java, 
line 4548
bq.  > 
<https://reviews.apache.org/r/4640/diff/3/?file=100315#file100315line4548>
bq.  >
bq.  >     private?

Ah yes, sorry, that should be private. Thanks!


- Ian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4640/#review6717
-----------------------------------------------------------


On 2012-04-05 18:52:50, Ian Varley wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4640/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-04-05 18:52:50)
bq.  
bq.  
bq.  Review request for hbase.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Fix for persistence of scan metrics when the scanner doesn't run to 
exhaustion.
bq.  
bq.  
bq.  This addresses bug HBASE-5717.
bq.      https://issues.apache.org/jira/browse/HBASE-5717
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    /src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java 
1309585 
bq.    /src/main/java/org/apache/hadoop/hbase/client/ClientScanner.java 1309585 
bq.  
bq.  Diff: https://reviews.apache.org/r/4640/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Altered the scan metrics unit test to show this problem (now fails without 
changes to ClientScanner.java).
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ian
bq.  
bq.


                
> Scanner metrics are only reported if you get to the end of a scanner
> --------------------------------------------------------------------
>
>                 Key: HBASE-5717
>                 URL: https://issues.apache.org/jira/browse/HBASE-5717
>             Project: HBase
>          Issue Type: Bug
>          Components: client, metrics
>            Reporter: Ian Varley
>            Priority: Minor
>         Attachments: ClientScanner_HBASE_5717-v2.patch, 
> ClientScanner_HBASE_5717-v3.patch, ClientScanner_HBASE_5717.patch
>
>   Original Estimate: 4h
>  Remaining Estimate: 4h
>
> When you turn on Scanner Metrics, the metrics are currently only made 
> available if you run over all records available in the scanner. If you stop 
> iterating before the end, the values are never flushed into the metrics 
> object (in the Scan attribute).
> Will supply a patch with fix and test.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to