IntelLabs / IntelLabs/vdms

Add FV and Blobs not cleaned up on failed transactions in query handle

Open
#189 1 comment 0 reactions 0 assignees View on GitHub
Bug Priority: Backlog
Dominant language
C++
Stars
93
Forks
34
PR merge metrics
No merged PRs in 30d

Description

**Describe the bug**
Currently the query cleanup logic PMGD query handling class does not include removal of stored FVs and blobs for failed transactions, meaning there is a risk of space leakage when a VDMS transaction is rolled back.

**To Reproduce**

Difficult to deterministically reproduce at it requires a transaction failure at the database level for PMGD, however the internal logic does not account for FV or Blob removal so its an obvious risk point.

**Expected behavior**
In the event an internal error that triggers a rollback, the written data should also be removed.

Currently this is an unlikely case, but we should not lose track of it.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.