Skip to content
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.

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

Add some additional logging to StorageService#drain #585

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

rhuffy
Copy link
Contributor

@rhuffy rhuffy commented Nov 21, 2024

This PR bumps some existing log statements from Debug to Info, and adds some additional log statements in the Drain codepath.

The intention here is to get better signal as to which parts of the Drain process take the most time.

@nickbar01234 nickbar01234 self-requested a review November 21, 2024 23:17
Copy link
Contributor

@nickbar01234 nickbar01234 left a comment

Choose a reason for hiding this comment

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

LGTM

@nickbar01234 nickbar01234 self-requested a review November 21, 2024 23:17
@rhuffy rhuffy merged commit afa0513 into palantir-cassandra-2.2.18 Nov 21, 2024
6 checks passed
@rhuffy rhuffy deleted the rh/log-draining branch November 21, 2024 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants