diff --git a/opencensus/exporters/stats/stackdriver/internal/stackdriver_exporter.cc b/opencensus/exporters/stats/stackdriver/internal/stackdriver_exporter.cc index 6af5ca42..099e5d2c 100644 --- a/opencensus/exporters/stats/stackdriver/internal/stackdriver_exporter.cc +++ b/opencensus/exporters/stats/stackdriver/internal/stackdriver_exporter.cc @@ -166,9 +166,11 @@ bool Handler::MaybeRegisterView( } // namespace // static -void StackdriverExporter::Register(const StackdriverOptions& opts) { +grpc::Status StackdriverExporter::Register(const StackdriverOptions& opts) { opencensus::stats::StatsExporter::RegisterPushHandler( absl::WrapUnique(new Handler(opts))); + // TODO: Propagate errors to caller. For now we just return OK. + return grpc::Status::OK; } // static, DEPRECATED diff --git a/opencensus/exporters/stats/stackdriver/stackdriver_exporter.h b/opencensus/exporters/stats/stackdriver/stackdriver_exporter.h index bbe70ac6..a071d04c 100644 --- a/opencensus/exporters/stats/stackdriver/stackdriver_exporter.h +++ b/opencensus/exporters/stats/stackdriver/stackdriver_exporter.h @@ -17,6 +17,7 @@ #include +#include #include "absl/base/macros.h" #include "absl/strings/string_view.h" #include "absl/time/time.h" @@ -42,8 +43,9 @@ struct StackdriverOptions { // Stackdriver. StackdriverExporter is thread-safe. class StackdriverExporter { public: - // Registers the exporter. - static void Register(const StackdriverOptions& opts); + // Registers the exporter and returns OK on success or another Status code + // otherwise. + static grpc::Status Register(const StackdriverOptions& opts); // TODO: Retire this: ABSL_DEPRECATED( diff --git a/opencensus/exporters/trace/stackdriver/internal/stackdriver_exporter.cc b/opencensus/exporters/trace/stackdriver/internal/stackdriver_exporter.cc index f0b36535..6f1e17c0 100644 --- a/opencensus/exporters/trace/stackdriver/internal/stackdriver_exporter.cc +++ b/opencensus/exporters/trace/stackdriver/internal/stackdriver_exporter.cc @@ -271,12 +271,14 @@ void Handler::Export( } // namespace // static -void StackdriverExporter::Register(const StackdriverOptions& opts) { +grpc::Status StackdriverExporter::Register(const StackdriverOptions& opts) { auto channel = ::grpc::CreateCustomChannel( kGoogleStackdriverTraceAddress, ::grpc::GoogleDefaultCredentials(), ::opencensus::common::WithUserAgent()); ::opencensus::trace::exporter::SpanExporter::RegisterHandler( absl::make_unique(opts, channel)); + // TODO: Propagate errors to caller. For now we just return OK. + return grpc::Status::OK; } // static, DEPRECATED diff --git a/opencensus/exporters/trace/stackdriver/stackdriver_exporter.h b/opencensus/exporters/trace/stackdriver/stackdriver_exporter.h index 0fcea9b9..1327a94d 100644 --- a/opencensus/exporters/trace/stackdriver/stackdriver_exporter.h +++ b/opencensus/exporters/trace/stackdriver/stackdriver_exporter.h @@ -17,6 +17,7 @@ #include +#include #include "absl/base/macros.h" #include "absl/strings/string_view.h" #include "absl/time/time.h" @@ -35,8 +36,9 @@ struct StackdriverOptions { class StackdriverExporter { public: - // Registers the exporter. - static void Register(const StackdriverOptions& opts); + // Registers the exporter and returns OK on success or another Status code + // otherwise. + static grpc::Status Register(const StackdriverOptions& opts); // TODO: Retire this: ABSL_DEPRECATED(