Skip to content

Benchmark and send telemetry events for indexing and resolution#4074

Merged
vinistock merged 1 commit intorubydex_adoption_feature_branchfrom
vs_add_telemetry_for_indexing_and_resolving
Apr 23, 2026
Merged

Benchmark and send telemetry events for indexing and resolution#4074
vinistock merged 1 commit intorubydex_adoption_feature_branchfrom
vs_add_telemetry_for_indexing_and_resolving

Conversation

@vinistock
Copy link
Copy Markdown
Member

Motivation

An important part after adopting Rubydex will be tracking performance of indexing and resolution (both full and incremental). This PR adds a benchmark telemetry event for all of these, so that we can easily spot performance regressions.

Implementation

The idea is to add a helper that benchmarks and automatically creates the telemetry event for us. I reused the ruby_lsp.response_time metric to avoid having to create a new one.

Automated Tests

Just a couple of corrections since we now push a new message to the outgoing queue.

@vinistock vinistock requested review from alexcrocha and st0012 April 22, 2026 19:27
@vinistock vinistock self-assigned this Apr 22, 2026
@vinistock vinistock requested a review from a team as a code owner April 22, 2026 19:27
@vinistock vinistock added server This pull request should be included in the server gem's release notes other Changes that aren't bugfixes, enhancements or breaking changes labels Apr 22, 2026
@vinistock vinistock merged commit 328b3db into rubydex_adoption_feature_branch Apr 23, 2026
38 checks passed
@vinistock vinistock deleted the vs_add_telemetry_for_indexing_and_resolving branch April 23, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

other Changes that aren't bugfixes, enhancements or breaking changes server This pull request should be included in the server gem's release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants