Skip to content

Move RelationalDB to spacetimedb-engine crate.#5113

Open
Shubham8287 wants to merge 17 commits into
masterfrom
shub/engine
Open

Move RelationalDB to spacetimedb-engine crate.#5113
Shubham8287 wants to merge 17 commits into
masterfrom
shub/engine

Conversation

@Shubham8287

@Shubham8287 Shubham8287 commented May 25, 2026

Copy link
Copy Markdown
Contributor

Description of Changes

Moves RelationalDB and related database code into a new spacetimedb-engine crate.
The main motivation is to tighten dependency control around the engine layer and isolate RelationalDB
behind a crate boundary.

  • Majority of this PR is code-motion.
  • Removes direct production dependence on tokio from spacetimedb-engine.
  • Keeps tokio only as a dev-dependency for test-only code in spacetimedb-engine.
  • This is intended to be a structural refactor only and should not result in any functional change in
    production.
  • Adds a CI check to ensure spacetimedb-engine continues to compile in simulation mode

API and ABI breaking changes

NA

Expected complexity level and risk

1.5.

Testing

Existing tests should be enough.

@cloutiertyler

Copy link
Copy Markdown
Contributor

Why is "Expected complexity level and risk" unspecified? Could you please update. I assume you're saying it's low risk because it's just mechanical?

@Shubham8287

Copy link
Copy Markdown
Contributor Author

Why is "Expected complexity level and risk" unspecified? Could you please update. I assume you're saying it's low risk because it's just mechanical?.

No, I would have come to back it, when asked someone to review it. It is draft rn. (I should have marked).

@Shubham8287 Shubham8287 marked this pull request as draft May 29, 2026 17:59
Base automatically changed from shub/sim to master June 4, 2026 15:46
@Shubham8287 Shubham8287 force-pushed the shub/engine branch 4 times, most recently from 312d9d7 to 59e77cf Compare June 9, 2026 12:30
Comment thread crates/durability/Cargo.toml
@Shubham8287 Shubham8287 marked this pull request as ready for review June 10, 2026 13:46
@Shubham8287 Shubham8287 requested a review from jsdt June 10, 2026 13:46
Comment thread crates/engine/src/relational_db.rs
@Shubham8287 Shubham8287 changed the base branch from shub/runtime-base to master June 15, 2026 09:29
pull Bot pushed a commit to Abaso007/SpacetimeDB that referenced this pull request Jun 15, 2026
…lockworklabs#5255)

# Description of Changes

Runtime specific changes from
clockworklabs#5113.

- `spacetimedb-runtime` enforces exactly one feature at compile time:
`tokio` or `simulation`. It rejects both-enabled and neither-enabled
builds explicitly.

- This also keeps `tokio::sync` re-exported from runtime, look at code
comment for reasoning.

# API and ABI breaking changes

NA

# Expected complexity level and risk
1
@Shubham8287 Shubham8287 enabled auto-merge June 15, 2026 15:08
@Shubham8287 Shubham8287 disabled auto-merge June 15, 2026 17:30
@Shubham8287 Shubham8287 enabled auto-merge June 16, 2026 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants