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

Josh Elser commented on HBASE-13078:
------------------------------------

Thanks for taking a look, Nick.

bq. a smoke test like this one that can be run to validate tracing is 
configured correctly cluster-wide (ie, and spans are coming out of both hbase 
and hdfs). Maybe the latter is unreasonable?

Difficult, perhaps. The instrumentation in HDFS will (continue to) be a moving 
target, I imagine. It might be better to get good tests in HDFS to ensure that 
tracing occurs where desired. If we can guarantee that spans in HDFS make it to 
the SpanReceiver, we shouldn't need to retest here in HBase.

bq. a medium or large test that verifies a single traced invocation with 
everything enabled results in spans logged (and perhaps some validation thereof)

Cool, I'll open up another issue for this. Opinions/thoughts on what this would 
look like are suggested/welcome. At a minimum, I can peruse the usages and try 
to tailor a test around that.

> IntegrationTestSendTraceRequests is a noop
> ------------------------------------------
>
>                 Key: HBASE-13078
>                 URL: https://issues.apache.org/jira/browse/HBASE-13078
>             Project: HBase
>          Issue Type: Test
>          Components: integration tests
>            Reporter: Nick Dimiduk
>            Assignee: Josh Elser
>            Priority: Critical
>             Fix For: 2.0.0, 1.1.0, 0.98.13, 1.0.2
>
>         Attachments: HBASE-13078-0.98-removal.patch, 
> HBASE-13078-0.98-v1.patch, HBASE-13078-v1.patch, HBASE-13078.patch
>
>
> While pair-debugging with [~jeffreyz] on HBASE-13077, we noticed that 
> IntegrationTestSendTraceRequests doesn't actually assert anything. This test 
> should be converted to use a mini cluster, setup a POJOSpanReceiver, and then 
> verify the spans collected.



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

Reply via email to