elastic / elastic/apm-agent-java

Add Span#deactivate to the public API

オープン
#780 コメント 4 件 リアクション 0 件 担当者 0 名 GitHub で見る
api
主要言語
Java
スター
593
フォーク
337
平均マージ
1日 13時間
マージ済み PR(30日)
25

説明

Currently, the public API does not allow for `deactivation` of active Transactions and Spans arbitrarily. The only way to deactivate is by keeping the `Scope` reference returned from the `activate` API.

An example where this would be required - [this discuss topic](https://discuss.elastic.co/t/spring-kafka-instrumentation-kafkalistenerendpointcontainer-mono-thread/193810).

I also found this limiting when implemented [this context propagation example](https://github.com/elastic/apm-agent-java/blob/24f60f9fcef59c58ad99d848efc28f14d6c772d1/apm-agent-plugins/apm-api-plugin/src/test/java/co/elastic/apm/api/BlockingQueueContextPropagationTest.java)

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

公開されている Span API と Scope API から始め、apm-agent-plugins/apm-api-plugin/src/test/java/co/elastic/apm/api/BlockingQueueContextPropagationTest.java を読んでください。既存のアクティベーションおよびデアクティベーションの動作を、context-propagation の例およびリンク先の Spring Kafka のユースケースと比較してください。公開 API が任意の Span のデアクティベーションをサポートし、テストで新しい動作をカバーできれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
api, backend-api-design
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。