orioledb / orioledb / 23907624522
90%
main: 90%

LAST BUILD BRANCH: harden-checkpoint-no-test
DEFAULT BRANCH: main
Ran
Jobs 1
Files 92
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 90.205% (-0.08%) from 90.287%
23907624522

push

github

za-arthur 
recovery: simplify WAL reader

Container flags (WAL_CONTAINER_HAS_XACT_INFO,
WAL_CONTAINER_HAS_ORIGIN_INFO)
are now parsed inline in parse_wal_container() directly into two new
fields on WalReaderState (xactTime, xactXid) instead of going through
a descriptor table, a per-flag helper function, and an on_flag callback.

All three call sites the separate *_wal_check_version and
*_wal_container_info callbacks are merged:

  - replay_wal_containter_info() is folded into
replay_wal_check_version();
    recoveryHeapTransactionId is assigned there from r->xactXid.
  - decode_wal_containter_info() is removed; has_origin is dropped from
    DecodeWalDescCtx and replaced by a direct rec->origin_id check in
    decode_wal_record().
  - recovery_wal_containter_info() is removed; the stops hook reads
    xactTime/xactXid from WalReaderState after parse_wal_container()
    returns.

g_wal_descs[] is rewritten using designated initialisers with a
per-entry
comment on the opening brace, matching the style of sysTreesMeta in
src/catalog/sys_trees.c. Each .parse field is now a direct function
reference that LSP can follow with go-to-definition.

The ORIOLE_WAL_RECORDS X-macro and the OrioleWalRecType enum generated
from it are removed as they had no remaining consumers. WAL_REC_*
constants are now plain #defines.

Block comments in parse_wal_container() and the removed
helper functions are replaced with concise prose.

45597 of 50548 relevant lines covered (90.21%)

49444458.54 hits per line

Coverage Regressions

Lines Coverage ∆ File
174
90.28
 -0.75% orioledb/src/recovery/recovery.c
81
78.35
 0.63% orioledb/src/orioledb.c
46
89.88
 0.49% orioledb/src/recovery/logical.c
17
91.32
 0.83% orioledb/src/recovery/wal.c
17
84.11
 -0.78% orioledb/src/recovery/wal_reader.c
16
90.35
 -1.59% orioledb/src/btree/scan.c
6
90.98
 -0.19% orioledb/src/checkpoint/checkpoint.c
5
92.02
 -1.33% orioledb/src/btree/merge.c
4
85.93
 -0.21% orioledb/src/btree/io.c
4
91.1
 -0.6% orioledb/src/s3/headers.c
3
85.26
 -0.63% orioledb/src/catalog/sys_trees.c
3
93.77
 -0.07% orioledb/src/transam/undo.c
2
88.77
 -0.23% orioledb/src/transam/oxid.c
1
71.11
 0.25% orioledb/src/btree/fastpath.c
1
97.87
 -0.13% orioledb/src/btree/page_chunks.c
1
88.9
 -0.1% orioledb/src/btree/undo.c
1
91.2
 -0.04% orioledb/src/catalog/ddl.c
1
97.28
 -0.09% orioledb/src/tableam/descr.c
1
86.32
 -1.05% orioledb/src/workers/bgwriter.c
Jobs
ID Job ID Ran Files Coverage
1 23907624522.1 92
90.21
 GitHub Action Run
Source Files on build 23907624522