perf: avoid creating exceptions when no hook implements the error stage
Los mantenedores suelen responder en 1 día
Evaluación
Este issue todavía no se ha evaluado.
Descripción
When no provider is set (or the provider is NOT_READY/FATAL), evaluateFlag throws an exception internally to reach its catch block. Similarly, for provider-returned errors, we create an exception to pass to error hooks, even if no hook implements error.
The ProviderNotReadyError stack trace is suppressed, but APM agents that instrument exceptions can still capture their own, which adds overhead to every evaluation.
Proposal:
- don't throw for
NOT_READY/FATAL; build the error details directly - only create the exception if an attached hook actually overrides
error(detected once per hook class)
Results (callers see the same errorCode, reason and value) are unchanged. Hooks that don't override error keep their no-op default, and the SDK just skips calling it.
- Lenguaje dominante
- Java
- Estrellas
- 128
- Forks
- 60
- Merge medio
- 19 h 11 min
- PR fusionados (30 d)
- 19
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de open-feature/java-sdk
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
open-feature/java-sdk#2019 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
open-feature/java-sdk#2053 ·
Los mantenedores suelen responder en 1 día
-
Spec v0.9.0 complianceAbiertov0.9.0
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
open-feature/java-sdk#1999 ·
Los mantenedores suelen responder en 1 día
-
on setProvider call, shutting down the previous provider should happen before creating the new oneAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 50/100
open-feature/java-sdk#1934 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
[Multi-provider] Gaps identified relative to js-sdk reference implementationPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertomulti-provider
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
open-feature/java-sdk#1882 ·
Los mantenedores suelen responder en 1 día
Todos los issues de open-feature/java-sdk
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
utopia-rise/godot-jvm#1004 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
spring-projects/spring-grpc#442 ·
-
Expose numberOfPermits in RateLimiterEvent.toString() and the ratelimiterevents actuator DTOPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
resilience4j/resilience4j#2547 ·
Los mantenedores suelen responder en 9 días
-
Clock.MakeDate continues execution and returns a rolled-over instant after dispatching error on invalid datePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
mit-cml/appinventor-sources#4155 ·
Los mantenedores suelen responder en 1 día