Skip to content

refactor: support closing all connections to a db in Rust#51

Open
onehumandev wants to merge 1 commit into
silvermine:masterfrom
onehumandev:onehumandev/close_invocation
Open

refactor: support closing all connections to a db in Rust#51
onehumandev wants to merge 1 commit into
silvermine:masterfrom
onehumandev:onehumandev/close_invocation

Conversation

@onehumandev

Copy link
Copy Markdown

Previously, the only way to close the connection to a database was through a command invocation, which is only available through IPC.

We need the ability to close any open connections to a db in Rust.

For example if we are performing file operations (such as deleting and recreating the db) from the Rust layer, we need to first ensure all connections are closed before taking any further action.

This simple refactor breaks the logic to close any open connections into a public function callable from Rust.

The logic is unchanged.

@onehumandev

Copy link
Copy Markdown
Author

c. @jjhafer @velocitysystems

Comment thread src/lib.rs Outdated
Comment thread src/lib.rs Outdated
Comment thread src/lib.rs Outdated
@onehumandev onehumandev force-pushed the onehumandev/close_invocation branch 3 times, most recently from 5aabb86 to 648d309 Compare June 16, 2026 17:48
Previously, the only way to close the connection to a database was
through a command invocation, which is only available through IPC.

We need the ability to close any open connections to a db in Rust.

For example if we are performing file operations (such as deleting and
recreating the db) from the Rust layer, we need to first ensure all
connections are closed before taking any further action.

This simple refactor breaks the logic to close any open connections
into a public function callable from Rust.
@onehumandev onehumandev force-pushed the onehumandev/close_invocation branch from 648d309 to 99fe607 Compare June 16, 2026 17:51
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.

4 participants