Client interceptors depend on server Endpoint set about brave HOT 4 CLOSED

Hey @fedor57 ,

So far with all implementations we have the endpoint set as part of the server side integration code.
This does not mean however that we should fail with NullPointerException at client side when endpoint has not been set. This should get fixed.

I also agree that we should be able to start a span from server or client side. Currently it looks like we have a dependency on the server side.

The X-B3-Span-Name header was introduced by me and had the purpose of aligning client/server side snap names. If a span name was created at client side which was not easily derivable at server side passing the name and re-using it at server side was a way to solve this. However I'm thinking of abandoning this approach and choosing simple span names, see #59.

I want to spend some time soon on the work needed for brave 3.0 and I'll take these 2 issues + documentation into account.

from brave.

Thanx, Kristof. I have answered on service/span naming on #59, regarding the correct approach for clients:

If current request had no instrumentation but the current module had a Zipkin library attached with client requests instrumented we should start a span automatically. No point in having "cs" without parent-span-id and break 3 client calls into different traces.

But theoretically there is a case when it shouldn't. Imagine you have a testing utility that fires 1000 requests from logs from different clients.

If we autostart the span it should start at least at the beginning of the first call and end at least at the end of the last.

But also we should be able to mark the beginning earlier and end later.

For the IP call we can use not Start/Done, but call some annotation method like AddAnnotation("Start"), AddAnnotation("End") - so those does not assume that current span exists or not, but makes sure that it will exist after the call. Or we can use AddAnnotation() without parameters that just checks the the current moment belongs to some span.

There is a problem with end of the span if no "ss" interceptor will be fired. So if I don't call Done - the trace is created but the current span is never finished. Maybe this to be considered - to autoclose span after timeout and mark it somehow?

from brave.

@fedor57 This issue should be fixed with brave 3.0.0-rc-1.

from brave.

Fixed. EndpointSubmitter does not exist anymore. No need to submit endpoint upfront.

from brave.