-
Notifications
You must be signed in to change notification settings - Fork 209
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
{173892112}: temptabl-ifying txnlist #4156
base: main
Are you sure you want to change the base?
Conversation
Recovery may run out of memory if there're too many txnid's to keep track of. This patch implements a temptable logic for txnlist that greatly reduces the memory footprint without sacrificing too much performance (partly because that log and bufferpool I/O would dominate performance in such a case). Signed-off-by: Rivers Zhang <hzhang320@bloomberg.net>
6411ba5
to
0c26d4a
Compare
Turning this on to run through RR & RM ... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 7/511 tests failed ⚠.
The first 10 failing tests are:
sc_truncate_lockorder_generated
sc_transactional
queuedb_rollover_noroll1_generated
queuedb_rollover
prepare
ddl_no_csc2
tunables
Passing rm: |
Signed-off-by: Rivers Zhang <hzhang320@bloomberg.net>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 3/511 tests failed ⚠.
The first 10 failing tests are:
queuedb_rollover
prepare
sp
Recovery may run out of memory if there're too many txnid's to keep track of. This patch implements a temptable logic for txnlist that greatly reduces the memory footprint without sacrificing too much performance (partly because that log and bufferpool I/O would dominate performance in such a case).