[
https://issues.apache.org/jira/browse/CASSANDRA-15143?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Semb Wever updated CASSANDRA-15143:
-------------------------------------------
Reviewers: Michael Semb Wever, Michael Semb Wever (was: Michael Semb Wever)
Michael Semb Wever, Michael Semb Wever
Status: Review In Progress (was: Patch Available)
||branch||circleci||asf jenkins testall||asf jenkins dtests||
|[trunk_15143|https://github.com/apache/cassandra/compare/trunk...thelastpickle:mck/trunk_15143]|[circleci|https://circleci.com/gh/thelastpickle/workflows/cassandra/tree/mck%2Ftrunk_15143]|[!https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-pipeline/48/badge/icon!|https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-pipeline/48/]|[!https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-dtest/722/badge/icon!|https://builds.apache.org/view/A-D/view/Cassandra/job/Cassandra-devbranch-dtest/722]|
> Remove assertion on file deletion to trigger failure policy
> -----------------------------------------------------------
>
> Key: CASSANDRA-15143
> URL: https://issues.apache.org/jira/browse/CASSANDRA-15143
> Project: Cassandra
> Issue Type: Bug
> Components: Local/SSTable
> Reporter: Stefan Podkowinski
> Assignee: Stefan Podkowinski
> Priority: Normal
>
> Deleting any files by using FileUtils.deleteWithConfirm() will involve
> checking for file existence using an assertion. If the assertion passes, any
> errors from the actual Files.delete() operation will get handled as a
> FSWriteError and invoke the DiskFailurePolicy. This will get us to a
> situation where only some cases of bad FS or disk issues will be handed by
> the DFP, while FS issues that may already cause the file.exists() assertion
> to fail, will only manifest as AssertionErrors. We should invoke DFP for the
> later as well and remove the assertion altogether.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
