api metrics missing toString/get implementation

[amouhadi]

All classes under io.opentelemetry.api.metrics are missing toString or any get method. e.g:(DoubleCounter, DoubleGaugeBuilder, DoubleHistogram, DoubleUpDownCounter, DoubleUpDownCounterBuilder, LongCounter, LongGaugeBuilder, LongUpDownCounter, LongUpDownCounterBuilder, ...)
After we use the Builder classes and set name, unit, description then there is no way to get the metric name or use toString to get the information about that api metrics. This is becoming more critical each time we need to debug/troubleshoot

[jkwatson]

Please submit a PR for this change! Thanks!

[mkorzeniewski]

May I be assigned to this issue?

[mkorzeniewski]

I can not see any way to add toString methods to members of the io.opentelemetry.api.metrics package. It contains interfaces which are contracts with no state to describe for the toString method.

I assumed the origin of the problem lies in the luck of toString method in io.opentelemetry.sdk.metrics package. Any instance of an AbstractInstrument has only default toString method which is not meaningful.

Now it should be possible to see the following fields name, unit, description.

[mkorzeniewski]

Does it resolve the problem somehow? Is a toString method needed for Builders? I believe it is always a way to create an instance and then call toString method

[jack-berg]

The change in #4833 does not add toString to the instrument builders. If you feel this is necessary please re-open.

[mkorzeniewski]

You can assign me to the new issue you will create.