[microprofile-dev] Issue with the Fault Tolerance 4.1 Open Telemetry met

[Andrew Rouse <ANROUSE@xxxxxxxxxx>]

Hi,

Just as we're coming up to release, we've had an issue raised about the
new Open Telemetry metrics added in Fault Tolerance 4.1:
https://github.com/eclipse/microprofile-fault-tolerance/issues/639

When we added the integration of Open Telemetry metrics, we used our
existing integration with MP Metrics as a guide. However, we didn't
realise that while both APIs have many of the same concepts (counters,
gauges, histograms etc.) the details between the two are quite
different and some of the choices we made for MP Metrics are not the
right choices for Open Telemetry metrics.

The changes being suggested aren't huge, but they're not trivial
either. I think I could have a PR ready in a few days, and then it
would take another few days to get the changes merged into OpenLiberty
and built before we could create another CCR.

Given that we're trying to get MP 7.0 released, I think the options
are:

1. Delay MP 7.0 again so that this issue can be addressed
2. Remove MP Fault Tolerance 4.1 from MP 7.0 (and include Fault
Tolerance 4.0 instead)
3. Release MP Fault Tolerance 4.1 in its current state (which would
require a breaking change to fix these problems in a later release)

The only change in MP Fault Tolerance 4.1 is the addition of these Open
Telemetry metrics, so personally I would prefer one of the first two
options, rather than releasing it with these issues.

Andrew Rouse


Unless otherwise stated above:

IBM United Kingdom Limited
Registered in England and Wales with number 741598
Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU