Skip to content

Breaking: remove deferred chained batch#51

Merged
vweevers merged 1 commit intov2from
no-deferred-chained-batch
Nov 14, 2022
Merged

Breaking: remove deferred chained batch#51
vweevers merged 1 commit intov2from
no-deferred-chained-batch

Conversation

@vweevers
Copy link
Copy Markdown
Member

@vweevers vweevers commented Nov 10, 2022

Removes the ability to use chained batch before the db is open, because due to its synchronous API, the problem of operations being deferred extends to hooks (#45). I.e. prewrite hook functions must also be functional if db.status is opening and that's an unnecessary burden for userland code.

@vweevers vweevers added the semver-major Changes that break backward compatibility label Nov 10, 2022
@vweevers vweevers added this to the 2.0.0 milestone Nov 10, 2022
@vweevers vweevers merged commit a89e5a0 into v2 Nov 14, 2022
@vweevers vweevers deleted the no-deferred-chained-batch branch November 14, 2022 21:54
vweevers added a commit that referenced this pull request Nov 15, 2022
Follow-up for #51. Found while testing v2 on `memory-level` (#57).
vweevers added a commit that referenced this pull request Nov 15, 2022
Follow-up for #51. Found while testing v2 on `memory-level` (#57).
vweevers added a commit that referenced this pull request Jan 27, 2024
vweevers added a commit that referenced this pull request Jan 27, 2024
Follow-up for #51. Found while testing v2 on `memory-level` (#57).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver-major Changes that break backward compatibility

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant