Skip to content

Storage: show in logs when crdb_internal.compact_engine_span is called #111954#118719

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
Manish4044:fix/111954
May 2, 2024
Merged

Storage: show in logs when crdb_internal.compact_engine_span is called #111954#118719
craig[bot] merged 1 commit intocockroachdb:masterfrom
Manish4044:fix/111954

Conversation

@Manish4044
Copy link
Copy Markdown
Contributor

@Manish4044 Manish4044 commented Feb 3, 2024

Fixes: #111954

Description
This PR introduces log entries to indicate when crdb_internal.compact_engine_span is called. This improvement aims to facilitate debugging escalations.

Changes
Adding log statements whenever crdb_internal.compact_engine_span builtin is called

Release Note: None
Epic: none

@Manish4044 Manish4044 requested a review from a team as a code owner February 3, 2024 05:46
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Feb 3, 2024

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

Thank you for contributing to CockroachDB. Please ensure you have followed the guidelines for creating a PR.

Before a member of our team reviews your PR, I have some potential action items for you:

  • Please ensure your git commit message contains a release note.
  • When CI has completed, please ensure no errors have appeared.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Copy Markdown
Member

cockroach-teamcity commented Feb 3, 2024

CLA assistant check
All committers have signed the CLA.

@blathers-crl blathers-crl bot added the O-community Originated from the community label Feb 3, 2024
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@RaduBerinde RaduBerinde requested review from a team and RaduBerinde May 1, 2024 00:51
@RaduBerinde
Copy link
Copy Markdown
Member

Sorry, I missed this. LGTM!

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented May 1, 2024

👎 Rejected by too few approved reviews

Copy link
Copy Markdown
Member

@yuzefovich yuzefovich left a comment

Choose a reason for hiding this comment

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

bors r=raduberinde

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @RaduBerinde)

@craig
Copy link
Copy Markdown
Contributor

craig bot commented May 2, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

O-community Originated from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants