ConfigProperties Javadoc does not match getDuration and getMap behavior
I maintainer di solito rispondono entro 2 giorni
Una pull request collegata è già stata integrata.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 94/100
- Tipo di issue
- Documentazione
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- java
- Ambito
- documentation
Direzione di ricerca
Inizia in sdk-extensions/autoconfigure-spi, in ConfigProperties.java, e rivedi il Javadoc dei tre metodi indicati. Confronta il comportamento documentato con DefaultConfigProperties.getDuration e con le asserzioni in ConfigPropertiesTest. Il lavoro è completato quando il Javadoc descrive accuratamente gli errori, le unità supportate e i valori restituiti senza modificare il comportamento o le firme.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- Java
- Stelle
- 2.5k
- Fork
- 1k
- Merge medio
- 2g 20h
- PR unite (30g)
- 62
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di open-telemetry/opentelemetry-java
-
OkHttpGrpcSender no longer logs retry attemptsForse già presa @thswlsqls l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
open-telemetry/opentelemetry-java#8966 ·
I maintainer di solito rispondono entro 2 giorni
-
JdkHttpSender passes CompletionException instead of the original exception to onErrorForse già presa @thswlsqls l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
open-telemetry/opentelemetry-java#8961 ·
I maintainer di solito rispondono entro 2 giorni
-
JdkHttpSender does not close the response body on unsupported Content-EncodingForse già presa @thswlsqls l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
open-telemetry/opentelemetry-java#8960 ·
I maintainer di solito rispondono entro 2 giorni
-
Prometheus exporter: exemplar filtered attributes override trace_id and span_id labelsForse già presa @thswlsqls l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
open-telemetry/opentelemetry-java#8958 ·
I maintainer di solito rispondono entro 2 giorni
-
UpstreamGrpcSender ignores a "Host" header when overriding the authorityForse già presa @thswlsqls l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
open-telemetry/opentelemetry-java#8956 ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di open-telemetry/opentelemetry-java
Issue simili
-
[Bug] dubbo.protocol.triple.max-message-size is only read from OS environment variablesForse già presa @CCCLL l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
hierynomus/sshj#1070 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
intellij-elixir/intellij-elixir#4492 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
-
[Server] OpenAPI 3.1 YAML specs generate Object fields: resolveToJSON drops every schema typeAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
quarkiverse/quarkus-openapi-generator#1796 · 1 commento ·