Skip to content

fix: preserve LFTJ admission errors and cleanup (#1369) - #2077

Open
justinjoy wants to merge 1 commit into
mainfrom
codex/1369-lftj
Open

justinjoy wants to merge 1 commit into
mainfrom
codex/1369-lftj

Conversation

@justinjoy

@justinjoy justinjoy commented Oct 4, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

This is a partial #1369 fix and does not close #1369.

Validation

Validated after rebasing onto current main at 74194e26e73be786d8094fe7027f6dda2bea05bd:

  • meson test -C build lftj lftj_integration memory_admission_join --print-errorlogs — all three tests passed.
  • uncrustify -c uncrustify.cfg --check wirelog/columnar/lftj.c wirelog/columnar/lftj.h wirelog/columnar/ops.c tests/test_lftj.c tests/test_lftj_integration.c — passed.
  • scripts/ci/check-threading-doc.sh — passed; all 222 atomic-site audit rows covered.
  • git diff --check origin/main...HEAD — passed.

Hosted CI for this rebased head is running.

Base automatically changed from codex/1369-audit to main October 4, 2026 23:49
@justinjoy
justinjoy force-pushed the codex/1369-lftj branch 2 times, most recently from df4f7a6 to 9b2ccac Compare October 5, 2026 00:56

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enforce memory admission at every allocation class; replace the join row cap and deprecate WIRELOG_JOIN_OUTPUT_LIMIT

1 participant