Skip to content

prepare for on-the-fly post-processing, no intermediate files - #1436

Merged
baltzell merged 7 commits into
developmentfrom
serial-honcho
Oct 7, 2026
Merged

prepare for on-the-fly post-processing, no intermediate files#1436
baltzell merged 7 commits into
developmentfrom
serial-honcho

Conversation

@baltzell

@baltzell baltzell commented Sep 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • store Schema instead of Bank, for multithread transparency
  • add serial pruning methods for memory management during inline post-processing
  • make serial stores volatile for thread visibility
  • remove synchronized, leave parallelism responsibilities to caller
    • previous and current usage is all serial, in Clas12Writer CLARA I/O writer service
  • add javadocs

@baltzell
baltzell marked this pull request as ready for review September 15, 2026 19:58
@baltzell baltzell mentioned this pull request Sep 15, 2026
Draft
@baltzell
baltzell enabled auto-merge (squash) September 19, 2026 01:46
@baltzell baltzell changed the title prepare for on-the-fly post-processing prepare for on-the-fly post-processing, no intermediate files Sep 19, 2026
@baltzell
baltzell force-pushed the serial-honcho branch 7 times, most recently from 78a8887 to 2b1cccf Compare September 30, 2026 17:20
@baltzell
baltzell requested a review from c-dilks as a code owner October 1, 2026 22:25
@baltzell
baltzell force-pushed the serial-honcho branch 2 times, most recently from 02f1bbb to 9ae0e11 Compare October 6, 2026 21:12
@baltzell baltzell added this to the Pass 3 milestone Oct 6, 2026
@baltzell
baltzell force-pushed the serial-honcho branch 3 times, most recently from e5c7c1b to 74b19b0 Compare October 7, 2026 18:04
@baltzell
baltzell merged commit 8d02fd4 into development Oct 7, 2026
24 checks passed
@baltzell
baltzell deleted the serial-honcho branch October 7, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raffaelladevita raffaelladevita raffaelladevita approved these changes
@c-dilks c-dilks Awaiting requested review from c-dilks

Assignees

@raffaelladevita raffaelladevita

Labels

None yet

Projects

None yet

Milestone

Pass 3

Development

Successfully merging this pull request may close these issues.

2 participants

@baltzell @raffaelladevita