opencensus-cpp icon indicating copy to clipboard operation
opencensus-cpp copied to clipboard

Change the Stackdriver exporters' Register() to return Status.

Open g-easy opened this issue 7 years ago • 1 comments

Make this non-breaking change to the API now, so we can use it to implement https://github.com/census-instrumentation/opencensus-cpp/issues/175 later.

g-easy avatar Nov 23 '18 04:11 g-easy

I think these functions should return grpc::Status because almost all of the possible errors come from the RPC stack.

The other kind of Status we have in OpenCensus is for describing the status of a span in our own data model, which I don't think is appropriate here.

Abseil may one day release its own Status type, but that hasn't happened yet.

g-easy avatar Dec 10 '18 02:12 g-easy