implement "b3 single" header format

[codefromthecrypt]

As discussed on openzipkin/b3-propagation#21 and first implemented here: https://github.com/openzipkin/brave/blob/master/brave/src/main/java/brave/propagation/B3SingleFormat.java https://github.com/openzipkin/brave/blob/master/brave/src/test/java/brave/propagation/B3SingleFormatTest.java

Let's support at least reading "b3" header from a single string, most commonly traceid-spanid-1
It would also be nice to support optionally writing this, especially in message providers or others with constrained environments.

Brave currently has a property like this, but its name could change with feedback:

    /**
     * When true, only writes a single {@link B3SingleFormat b3 header} for outbound propagation.
     *
     * <p>Use this to reduce overhead. Note: normal {@link Tracing#propagation()} is used to parse
     * incoming headers. The implementation must be able to read "b3" headers.
     */
    public Builder b3SingleFormat(boolean b3SingleFormat) {
      this.b3SingleFormat = b3SingleFormat;
      return this;
}

[fedj]

Hi @adriancole,

To clarify, what would be the priority of headers ? First check "b3" header and if not present, go for backward compatibility (checking "old" headers) ?

[codefromthecrypt]

yep cheapest check is to do "b3" first, the fallback to the existing logic.

…

On Thu, Aug 30, 2018 at 2:54 PM Jeanneret Pierre-Hugues ***@***.***> wrote: Hi @adriancole, To clarify, what would be the priority of headers ? First check "b3" header and if not present, go for backward compatibility (checking "old" headers) ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.