[
https://issues.apache.org/jira/browse/HDDS-8529?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Siyao Meng updated HDDS-8529:
-----------------------------
Description:
This is a follow-up to HDDS-8166.
Refactor {{OmSnapshotManager}} constructor to further reduce the initialization
steps (e.g. SnapDiff DB initialization, its column family creation) that are
done when snapshot feature is *disabled*.
See the block comment right above {{OmSnapshotManager}} constructor for more
details.
Original block comment:
{code}
/**
* TODO: [SNAPSHOT] HDDS-8529: Refactor the constructor in a way that when
* ozoneManager.isFilesystemSnapshotEnabled() returns false,
* no snapshot-related background job or initialization would run,
* except for applying previously committed Ratis transactions in e.g.:
* 1. {@link OMKeyPurgeRequest#validateAndUpdateCache}
* 2. {@link OMDirectoriesPurgeRequestWithFSO#validateAndUpdateCache}
*/
public OmSnapshotManager(OzoneManager ozoneManager) {
{code}
was:
This is a follow-up to HDDS-8166.
Refactor {{OmSnapshotManager}} constructor to further reduce the initialization
steps (e.g. SnapDiff DB initialization, its column family creation) that are
done when snapshot feature is *disabled*.
See the block comment right above {{OmSnapshotManager}} constructor for more
details.
cc [~hemantk]
> [Snapshot] Refactor OmSnapshotManager constructor to reduce initialization
> done when snapshot feature is disabled
> -----------------------------------------------------------------------------------------------------------------
>
> Key: HDDS-8529
> URL: https://issues.apache.org/jira/browse/HDDS-8529
> Project: Apache Ozone
> Issue Type: Sub-task
> Components: Snapshot
> Reporter: Siyao Meng
> Priority: Major
>
> This is a follow-up to HDDS-8166.
> Refactor {{OmSnapshotManager}} constructor to further reduce the
> initialization steps (e.g. SnapDiff DB initialization, its column family
> creation) that are done when snapshot feature is *disabled*.
> See the block comment right above {{OmSnapshotManager}} constructor for more
> details.
> Original block comment:
> {code}
> /**
> * TODO: [SNAPSHOT] HDDS-8529: Refactor the constructor in a way that when
> * ozoneManager.isFilesystemSnapshotEnabled() returns false,
> * no snapshot-related background job or initialization would run,
> * except for applying previously committed Ratis transactions in e.g.:
> * 1. {@link OMKeyPurgeRequest#validateAndUpdateCache}
> * 2. {@link OMDirectoriesPurgeRequestWithFSO#validateAndUpdateCache}
> */
> public OmSnapshotManager(OzoneManager ozoneManager) {
> {code}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
