Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

add a compaction filter long term case #378

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hicqu
Copy link
Contributor

@hicqu hicqu commented Mar 3, 2021

Signed-off-by: qupeng qupeng@pingcap.com

What problem does this PR solve?

Add a long term case for GC in Compaction Filter.

What is changed and how does it work?

The case has such features:

  • Has both long records and short records, which covers different code path for GC in compaction filter.
  • 3 SQLs are contianed in its workload: update/delete/insert.
  • 4 actions on TiKV side: MVCC_PUT(record), MVCC_PUT(index), MVCC_DELETE(record), MVCC_DELETE(index).
  • A simple transaction consistency checker during running, like bank2, but no transaction log tracing.

TODO list

  • Support to automatic run it on tipocket.

Check List

Tests

  • No code

Code changes

  • Has Go code change

Does this PR introduce a user-facing change?:

NONE

hicqu added 2 commits March 3, 2021 14:33
Signed-off-by: qupeng <qupeng@pingcap.com>
Signed-off-by: qupeng <qupeng@pingcap.com>
@mahjonp
Copy link
Contributor

mahjonp commented Mar 5, 2021

  • Support to automatic run it on tipocket.

This test case couldn't run on tipocket now since the dependencies are lack in the docker image, I plan to let each test case owns a Dockerfile so that dependencies and manifests(Lua, Sysbench, etc) can be contained on the split images.

Until then, this issue is resolved.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants