Skip to content

Fix BatchLogRecordProcessor default schedule_delay_millis to 1000ms per OTel spec #4998

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
xrmx merged 8 commits into from
Apr 3, 2026
Merged

Fix BatchLogRecordProcessor default schedule_delay_millis to 1000ms per OTel spec #4998

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
3979f16
Fix BatchLogRecordProcessor default schedule_delay_millis to 1000ms p…
Manvi2402 Mar 18, 2026
039ef13
Update OTEL_BLRP_SCHEDULE_DELAY docstring default to 1000ms and fix C…
Manvi2402 Mar 19, 2026
4a52e74
Apply suggestion from @xrmx
xrmx Mar 19, 2026
06eb065
Merge branch 'main' into fix/batch-log-processor-schedule-delay
xrmx Mar 20, 2026
73b03f3
Update CHANGELOG wording to 'may be' for export frequency
Manvi2402 Mar 28, 2026
c058b51
Merge branch 'fix/batch-log-processor-schedule-delay' of https://gith…
Manvi2402 Mar 28, 2026
96514cd
Apply suggestion from @xrmx
xrmx Apr 3, 2026
0ed243d
Merge branch 'main' into fix/batch-log-processor-schedule-delay
xrmx Apr 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## Unreleased

- Fix `BatchLogRecordProcessor` default `schedule_delay_millis` from 5000ms to 1000ms to comply with the OTel specification. Note: logs may be exported 5x more frequently by default (e.g. for users who don't explicitly set the `OTEL_BLRP_SCHEDULE_DELAY` env var).
([#4998](https://github.com/open-telemetry/opentelemetry-python/pull/4998))
- `opentelemetry-sdk`: Add `process` resource detector support to declarative file configuration via `detection_development.detectors[].process`
([#5001](https://github.com/open-telemetry/opentelemetry-python/pull/5001))
- `opentelemetry-sdk`: Add shared `_parse_headers` helper for declarative config OTLP exporters
Expand Down
2 changes: 1 addition & 1 deletion opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/export/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@
OtelComponentTypeValues,
)

_DEFAULT_SCHEDULE_DELAY_MILLIS = 5000
_DEFAULT_SCHEDULE_DELAY_MILLIS = 1000
Comment thread
xrmx marked this conversation as resolved.
_DEFAULT_MAX_EXPORT_BATCH_SIZE = 512
_DEFAULT_EXPORT_TIMEOUT_MILLIS = 30000
_DEFAULT_MAX_QUEUE_SIZE = 2048
Expand Down
2 changes: 1 addition & 1 deletion opentelemetry-sdk/src/opentelemetry/sdk/environment_variables/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@
.. envvar:: OTEL_BLRP_SCHEDULE_DELAY

The :envvar:`OTEL_BLRP_SCHEDULE_DELAY` represents the delay interval between two consecutive exports of the BatchLogRecordProcessor.
Default: 5000
Default: 1000
"""

OTEL_BLRP_EXPORT_TIMEOUT = "OTEL_BLRP_EXPORT_TIMEOUT"
Expand Down
6 changes: 3 additions & 3 deletions opentelemetry-sdk/tests/logs/test_export.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -582,7 +582,7 @@ def test_args_defaults(self):
log_record_processor._batch_processor._max_queue_size, 2048
)
self.assertEqual(
log_record_processor._batch_processor._schedule_delay, 5
log_record_processor._batch_processor._schedule_delay, 1
)
self.assertEqual(
log_record_processor._batch_processor._max_export_batch_size, 512
Expand Down Expand Up @@ -613,7 +613,7 @@ def test_args_env_var_value_error(self):
log_record_processor._batch_processor._max_queue_size, 2048
)
self.assertEqual(
log_record_processor._batch_processor._schedule_delay, 5
log_record_processor._batch_processor._schedule_delay, 1
)
self.assertEqual(
log_record_processor._batch_processor._max_export_batch_size, 512
Expand All @@ -639,7 +639,7 @@ def test_args_none_defaults(self):
log_record_processor._batch_processor._max_queue_size, 2048
)
self.assertEqual(
log_record_processor._batch_processor._schedule_delay, 5
log_record_processor._batch_processor._schedule_delay, 1
)
self.assertEqual(
log_record_processor._batch_processor._max_export_batch_size, 512
Expand Down
Loading
⚡