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

Jonathan Leavitt commented on HBASE-6524:
-----------------------------------------

bq. Doc is great. "(it would not be very difficult to remove this 
requirement)." What would this take?
Well to remove the client addendum we would have to instrument the entry points 
into hbase. 
I will try to find one good class to instrument as the entry point in to the 
system.  HTable.java might work, but it doesn't include all operations, and 
importantly does not include all of the steps of a 'get', bx getting the HTable 
is a big part, right? 
Maybe HTablePool.java would work. 

bq.  Can we do this for hdfs too yet? 
Not yet. I think Matt or Todd might be working on it. I have been looking into, 
but it does require all of the additional instrumentation to HDFS. 

bq. What about an example that enables trace over hdfs too while the Get is 
going on?
I'm not sure what you mean by this. 

bq. On doc in general, I came across this recent quote "Documentation is like 
sex: when it is good, it is very very good; and when it is bad, it is better 
than nothing."
Great quote. 
                
> Hooks for hbase tracing
> -----------------------
>
>                 Key: HBASE-6524
>                 URL: https://issues.apache.org/jira/browse/HBASE-6524
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Jonathan Leavitt
>            Assignee: Jonathan Leavitt
>             Fix For: 0.96.0
>
>         Attachments: 6524.addendum, 6524-v2.txt, 6524v3.txt, 
> createTableTrace.png, hbase-6524.diff
>
>
> Includes the hooks that use [htrace|http://www.github.com/cloudera/htrace] 
> library to add dapper-like tracing to hbase.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to