Refreshing via ContextRefresher pins virtual threads
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- java, spring-boot
- 領域
- backend
調査の方向性
ContextRefresher.refresh と refreshEnvironment から始め、続いて周辺の refresh フローを調べて、ほかに同期化されたセクションがないか確認します。仮想スレッドを使って /actuator/refresh パスを再現し、ロック動作を比較します。完了条件は、安全な環境およびスコープの同時 refresh を維持しながら、refresh 操作によって仮想スレッドがピン留めされなくなることです。
索引モデルが issue の本文から書いたものです。
説明
Describe the bug
With virtual threads enabled a call to /actuator/refresh pins virtual threads.
The ContextRefresher uses synchronized on both refresh and refreshEnvironment, so it seems they are not ready for use with virtual threads:
public synchronized Set<String> refresh() {
Set<String> keys = refreshEnvironment();
this.scope.refreshAll();
return keys;
}
public synchronized Set<String> refreshEnvironment() {
Map<String, Object> before = extract(this.context.getEnvironment().getPropertySources());
updateEnvironment();
Set<String> keys = changes(before, extract(this.context.getEnvironment().getPropertySources())).keySet();
this.context.publishEvent(new EnvironmentChangeEvent(this.context, keys));
return keys;
}
There could be more places of course
Are there plans to support virtual threads (with ReentrantLock)?
- 主要言語
- Java
- スター
- 750
- フォーク
- 749
- 平均マージ
- 19時間 58分
- マージ済み PR（30日）
- 23
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
spring-cloud/spring-cloud-commons のほかの issue
-
waiting-for-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
spring-cloud/spring-cloud-commons#1708 ·
メンテナーはふだん 1 日以内に返信
-
waiting-for-triage
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
spring-cloud/spring-cloud-commons#1543 ·
メンテナーはふだん 1 日以内に返信
-
`resetBeanToDefaults` transiently nulls nested `@ConfigurationProperties` objects, NPEing concurrent request threads対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbug
難易度 5/5 1週間以上 初心者へのやさしさ 48/100
spring-cloud/spring-cloud-commons#1727 · コメント 5 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement for team discussion
難易度 5/5 1週間以上 初心者へのやさしさ 42/100
spring-cloud/spring-cloud-commons#1676 ·
メンテナーはふだん 1 日以内に返信
-
waiting-for-triage
難易度 5/5 1週間以上 初心者へのやさしさ 38/100
spring-cloud/spring-cloud-commons#1675 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
spring-cloud/spring-cloud-commons の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
breaking change bug needs human review not ready priority medium size S
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
martin-francois/symphony-trello#1193 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
Mend: dependency security vulnerability
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100