Skip to content

Conversation

@swamirishi
Copy link
Contributor

@swamirishi swamirishi commented Mar 14, 2023

What changes were proposed in this pull request?

  1. We need to make sure that Snapshot RocksDB instance are opened with configuration option that turns of the auto-compaction.

  2. We open rocksDB for snapshot in RW for file filtering service. This is to delete some irrelevant SST files from the snapshot rocksDB manifest. We need to make sure that one of the configuration option in rocksDB option turns of Auto compaction.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-8102
https://issues.apache.org/jira/browse/HDDS-8122

How was this patch tested?

Unit Tests & Integration Tests

@aswinshakil aswinshakil added the snapshot https://issues.apache.org/jira/browse/HDDS-6517 label Mar 14, 2023
@smengcl smengcl requested review from aswinshakil and smengcl March 14, 2023 21:11
@swamirishi swamirishi changed the title HDDS-8122. Turning of auto compaction for sst-file-filtering service as well as garbage collection tasks HDDS-8122. Turning of auto compaction for sst-file-filtering service & Snapshot db open Mar 15, 2023
@swamirishi swamirishi changed the title HDDS-8122. Turning of auto compaction for sst-file-filtering service & Snapshot db open HDDS-8122. Turning of auto compaction for sst-file-filtering service & Snapshot diff Mar 15, 2023
# Conflicts:
#	hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/SstFilteringService.java
Copy link
Contributor

@smengcl smengcl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine with this approach.

Though I believe disableAutoCompactions == false by default, so we don't quite have to use Optional here.

@swamirishi
Copy link
Contributor Author

by d

In the case of DB Profile TEST we are disabling auto compaction, by default. We don't want it get overriden.

@smengcl
Copy link
Contributor

smengcl commented Mar 16, 2023

by d

In the case of DB Profile TEST we are disabling auto compaction, by default. We don't want it get overriden.

Ok. Makes sense. Thanks Swami!

@smengcl smengcl changed the title HDDS-8122. Turning of auto compaction for sst-file-filtering service & Snapshot diff HDDS-8122. [Snapshot] Disable RocksDB column family auto compaction in SstFilteringService Mar 16, 2023
@smengcl smengcl merged commit 6c18d95 into apache:master Mar 16, 2023
@smengcl
Copy link
Contributor

smengcl commented Mar 16, 2023

@swamirishi We are not quite disabling CF auto compaction in GC or SnapDiff yet IIUC. Thus the title change.

It can be added in #4407 or a new PR. cc @aswinshakil

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

snapshot https://issues.apache.org/jira/browse/HDDS-6517

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants