Skip to content

feat: improve observability#323

Open
yordis wants to merge 1 commit intocommanded:masterfrom
yordis:yordis/feat-add-telemetry
Open

feat: improve observability#323
yordis wants to merge 1 commit intocommanded:masterfrom
yordis:yordis/feat-add-telemetry

Conversation

@yordis
Copy link
Copy Markdown
Contributor

@yordis yordis commented Apr 14, 2026

  • Make EventStore behavior visible to consumers that need first-party telemetry instead of relying on adapters around it.
  • Align lazy stream observability with the actual batch reads that reach PostgreSQL.
  • Keep the telemetry namespace consistent with the EventStore application identity.

@yordis yordis marked this pull request as draft April 14, 2026 01:28
@yordis yordis force-pushed the yordis/feat-add-telemetry branch 6 times, most recently from 6c3b2ea to 2f808d4 Compare April 14, 2026 02:51
@yordis yordis marked this pull request as ready for review April 14, 2026 02:55
@yordis yordis force-pushed the yordis/feat-add-telemetry branch from 2f808d4 to 777c0ad Compare April 14, 2026 03:00
assert {:error, error} =
Appender.append(conn, 1, recorded_events, schema: schema, timeout: 0)

assert match?(%DBConnection.ConnectionError{}, error) or error == :query_canceled
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is an flaky test 😭

@yordis yordis force-pushed the yordis/feat-add-telemetry branch from 777c0ad to 698bac3 Compare April 15, 2026 00:37
@yordis yordis force-pushed the yordis/feat-add-telemetry branch 5 times, most recently from 82e7376 to 5266518 Compare April 19, 2026 05:12
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis yordis force-pushed the yordis/feat-add-telemetry branch from 5266518 to 3ea659c Compare April 19, 2026 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant