[
https://issues.apache.org/jira/browse/HDDS-14768?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Saketa Chalamchala updated HDDS-14768:
--------------------------------------
Description:
SnapshotCache cleanup might be unnecessarily strict.
The cleanup throws an `IllegalStateException` when it finds stale entries in
`pendingEvictionQueue` for snapshots that have already been removed from dbMap
(say when SnapshotPurge invalidates the entry).
It might be better for cleanup to log the stale entry proceed without failing
since this is not a correctness bug.
Also, invalidate should also remove the entry from `pendingEvictionQueue` to be
more consistent with bookkeeping.
was:
SnapshotCache cleanup might be unnecessarily strict.
The cleanup throws an `IllegalStateException` when it finds stale entries in
`pendingEvictionQueue` for snapshots that have already been removed from dbMap
(say when SnapshotPurge invalidates the entry).
It might be better for cleanup to log the stale entry proceed without failing
since this is not a correctness bug
> Log stale eviction entries in SnapshotCache cleanup instead of throwing an
> exception
> ------------------------------------------------------------------------------------
>
> Key: HDDS-14768
> URL: https://issues.apache.org/jira/browse/HDDS-14768
> Project: Apache Ozone
> Issue Type: Bug
> Components: OM
> Reporter: Saketa Chalamchala
> Assignee: Saketa Chalamchala
> Priority: Major
>
> SnapshotCache cleanup might be unnecessarily strict.
> The cleanup throws an `IllegalStateException` when it finds stale entries in
> `pendingEvictionQueue` for snapshots that have already been removed from
> dbMap (say when SnapshotPurge invalidates the entry).
> It might be better for cleanup to log the stale entry proceed without failing
> since this is not a correctness bug.
> Also, invalidate should also remove the entry from `pendingEvictionQueue` to
> be more consistent with bookkeeping.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
