Skip to content

service: fix 'occured' -> 'occurred' in DiskErrorsHandlerService messages#4742

Closed
SAY-5 wants to merge 1 commit into
apache:trunkfrom
SAY-5:fix-diskerrors-occured-typo
Closed

service: fix 'occured' -> 'occurred' in DiskErrorsHandlerService messages#4742
SAY-5 wants to merge 1 commit into
apache:trunkfrom
SAY-5:fix-diskerrors-occured-typo

Conversation

@SAY-5
Copy link
Copy Markdown

@SAY-5 SAY-5 commented Apr 16, 2026

Two messages in src/java/org/apache/cassandra/service/DiskErrorsHandlerService.java (lines 57, 62) read Exception occured while. Fixed to occurred. One is a logger.warn string, the other a ConfigurationException message. String-literal-only change.

…exception messages

Two messages in src/java/org/apache/cassandra/service/DiskErrorsHandlerService.java (lines 57, 62) read 'Exception occured while'. Fixed to 'occurred'. String-literal-only change.

Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
@pmcfadin
Copy link
Copy Markdown
Contributor

pmcfadin commented Apr 16, 2026

Thanks for the contribution, but the level of testing in our ecosystem is much higher than most. While this seems like a very simple change, it may actually break a test somewhere. Would you be willing to do a more general set of spell checks across all messages and validate that the changes don't break anything? Since this is touching a java file, we will require a dtest run to validate it didn't create regressions.

You can learn more about that here: https://cassandra.apache.org/_/development/testing.html

@pmcfadin pmcfadin added needs-committer Patch ready, waiting for a committer to review and merge tests and removed needs-committer Patch ready, waiting for a committer to review and merge labels Apr 16, 2026
@SAY-5
Copy link
Copy Markdown
Author

SAY-5 commented Apr 28, 2026

Closing — typo-only PR. Multiple maintainers across the OSS ecosystem have flagged my recent typo-sweep PRs as AI-generated spam (notably hashicorp/nomad#27855, hashicorp/hcl#794, argo-cd reviewers). I should have caught this pattern sooner. Apologies for the noise.

@SAY-5 SAY-5 closed this Apr 28, 2026
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.

3 participants