Skip to content

Commit

Permalink
fix: missing link to nlohmann_json (#1390)
Browse files Browse the repository at this point in the history
  • Loading branch information
@leodido
leodido committed May 13, 2022
1 parent 88e23ad commit 4b69a10
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion exporters/elasticsearch/CMakeLists.txt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@ target_include_directories(

target_link_libraries(
opentelemetry_exporter_elasticsearch_logs
PUBLIC opentelemetry_trace opentelemetry_logs opentelemetry_http_client_curl)
PUBLIC opentelemetry_trace opentelemetry_logs opentelemetry_http_client_curl
nlohmann_json::nlohmann_json)

install(
TARGETS opentelemetry_exporter_elasticsearch_logs
Expand Down

1 comment on commit 4b69a10

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'OpenTelemetry-cpp sdk Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 4b69a10 Previous: 88e23ad Ratio
BM_BaselineBuffer/2 9096236.228942871 ns/iter 2652756.690979004 ns/iter 3.43
BM_LockFreeBuffer/1 3423680.543899536 ns/iter 384407.8213786791 ns/iter 8.91
BM_LockFreeBuffer/2 3808616.3597205505 ns/iter 1158119.6784973145 ns/iter 3.29

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.