Skip to content

Commit cfd7cd7

Browse files
committed
Fix windows build
Signed-off-by: can <[email protected]>
1 parent 470955b commit cfd7cd7

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

.bazelrc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,8 @@ test --test_env=RAY_USAGE_STATS_REPORT_URL="http://127.0.0.1:8000"
4747
test --test_env=RAY_ENABLE_WINDOWS_OR_OSX_CLUSTER="1"
4848
# This is needed for some core tests to run correctly
4949
build:windows --enable_runfiles
50+
# This is needed since Windows has long file path issues and command length limits
51+
build:windows --features=compiler_param_file
5052
# TODO(mehrdadn): Revert the "-\\.(asm|S)$" exclusion when this Bazel bug
5153
# for compiling assembly files is fixed on Windows:
5254
# https://github.com/bazelbuild/bazel/issues/8924

src/ray/telemetry/open_telemetry_metric_recorder.cc

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515

1616
#include <opentelemetry/exporters/otlp/otlp_grpc_metric_exporter.h>
1717
#include <opentelemetry/metrics/provider.h>
18+
#include <opentelemetry/nostd/shared_ptr.h>
1819
#include <opentelemetry/sdk/metrics/export/periodic_exporting_metric_reader.h>
1920
#include <opentelemetry/sdk/metrics/instruments.h>
2021

@@ -52,7 +53,9 @@ void OpenTelemetryMetricRecorder::RegisterGrpcExporter(
5253
OpenTelemetryMetricRecorder::OpenTelemetryMetricRecorder() {
5354
// Default constructor
5455
meter_provider_ = std::make_shared<opentelemetry::sdk::metrics::MeterProvider>();
55-
opentelemetry::metrics::Provider::SetMeterProvider(meter_provider_);
56+
opentelemetry::metrics::Provider::SetMeterProvider(
57+
opentelemetry::nostd::shared_ptr<opentelemetry::metrics::MeterProvider>(
58+
meter_provider_));
5659
}
5760

5861
} // namespace telemetry

0 commit comments

Comments
 (0)