ConfigProperties Javadoc does not match getDuration and getMap behavior
メンテナーはふだん 2 日以内に返信
関連するプルリクエストがすでにマージされています。
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 94/100
- issue の種類
- ドキュメント
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- java
調査の方向性
sdk-extensions/autoconfigure-spi の ConfigProperties.java から始め、名前が示されている 3 つのメソッドの Javadoc を確認してください。ドキュメントに記載された動作を DefaultConfigProperties.getDuration および ConfigPropertiesTest のアサーションと比較してください。動作やシグネチャを変更せずに、Javadoc がエラー、サポートされている単位、戻り値を正確に説明していれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Is your feature request related to a problem? Please describe.
The Javadoc of the stable ConfigProperties SPI does not match how the default implementation behaves:
getDuration(String)says it returnsnullif the property "has a wrong type". Its own@throwssays malformed values throw, andDefaultConfigProperties.getDurationthrowsConfigurationExceptionfor a bad number or unit (asserted inConfigPropertiesTest). A custom implementation that follows the first sentence would silently fall back to defaults.- The
getDuration(String)unit list omitsnsandus, whichDefaultConfigPropertiesaccepts. - The
@returnofgetDuration(String, Duration)andgetMap(String, Map)says "a Double-valued configuration property", copied fromgetDouble(String, double).
Describe the solution you'd like
Correct the three points above in ConfigProperties.java. Javadoc only; no behavior or signature change.
Describe alternatives you've considered
N/A
Spec alignment
N/A — Java SPI documentation only.
Additional context
- Affected module:
:sdk-extensions:autoconfigure-spi - Artifact stability: stable
- Backward compatible: yes
- 主要言語
- Java
- スター
- 2.5k
- フォーク
- 1k
- 平均マージ
- 2日 22時間
- マージ済み PR(30日)
- 59
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
open-telemetry/opentelemetry-java のほかの issue
-
OkHttpGrpcSender no longer logs retry attempts対応中かも @thswlsqls が 10 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
open-telemetry/opentelemetry-java#8966 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
JdkHttpSender passes CompletionException instead of the original exception to onError対応中かも @thswlsqls が 10 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
open-telemetry/opentelemetry-java#8961 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
Fix `otel.sdk.log.created` counting semantics対応中かも @anuraaga が 84 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
open-telemetry/opentelemetry-java#8688 ·
メンテナーはふだん 2 日以内に返信
-
Bug
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
open-telemetry/opentelemetry-java#8982 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
難易度 4/5 1〜2日 初心者へのやさしさ 56/100
open-telemetry/opentelemetry-java#8980 ·
メンテナーはふだん 2 日以内に返信
open-telemetry/opentelemetry-java の issue をすべて見る
似ている issue
-
Component - Build
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
navikt/mock-oauth2-server#1149 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
oracle/graalvm-reachability-metadata#11318 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
Stirling-Tools/Stirling-PDF#8709 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area/config
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
quarkusio/quarkus#57910 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信