Skip to content

SequenceBuilder: add .Reset() method and use it during merge#19567

Open
AskAlexSharov wants to merge 8 commits intomainfrom
alex/ef_reset_34
Open

SequenceBuilder: add .Reset() method and use it during merge#19567
AskAlexSharov wants to merge 8 commits intomainfrom
alex/ef_reset_34

Conversation

@AskAlexSharov
Copy link
Collaborator

@AskAlexSharov AskAlexSharov commented Mar 2, 2026

Problem:
Screenshot 2026-03-02 at 17 20 26

In PR:

  • re-use 1 Builder object for many EF merges and many EF builds (in II.mergeFiles)
  • moved Merge method from Reader to Builder
  • moved reusable iterators for Merge method inside Builder object
  • “alloc=2.7g sys=5.4g” -> “alloc=2.7g sys=2.7g” (during large .rf file merge)

@AskAlexSharov AskAlexSharov marked this pull request as ready for review March 2, 2026 10:20
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.

2 participants