Skip to content

Commit

Permalink
Update 4290-logger-enabled.md
Browse files Browse the repository at this point in the history
  • Loading branch information
pellared authored Dec 16, 2024
1 parent 5a76201 commit 5046347
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions oteps/logs/4290-logger-enabled.md
Original file line number Diff line number Diff line change
Expand Up @@ -143,8 +143,8 @@ for different log record destinations.
## Alternatives

There was a [proposal](https://github.com/open-telemetry/opentelemetry-specification/issues/4207#issuecomment-2501688210)
to make the `LoggerConfig` to support also dynamic evaluation
instead of supporting only declarative configuration.
to make the `LoggerConfig` to support dynamic evaluation
instead of supporting only static configuration.
However, it seems that the purpose of the `LoggerConfig` is static configuration.
Moreover, both (5) and (6) use cases are coupled to log record processing,
therefore it seems more straighforward to extend `LogRecordProcessor`.
Expand Down

0 comments on commit 5046347

Please sign in to comment.