Skip to content

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

Open
SAY-5 wants to merge 1 commit intoapache:trunkfrom
SAY-5:fix-diskerrors-occured-typo
Open

service: fix 'occured' -> 'occurred' in DiskErrorsHandlerService messages#4742
SAY-5 wants to merge 1 commit intoapache: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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants