Use chdb npm runtime for local mode - #791
Conversation
|
Compatibility note from a local fixture test: I generated a checkpoint with Maple’s previous chdb-core
Then I changed only the fixture manifest’s This suggests the tested backup bytes are compatible with 26.7.2.1. For checkpoint restore, we may want to replace the strict |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
I generally wouldn’t mind using the npm package directly, but this breaks It also adds |
Summary
This changes Maple Local's embedded chDB integration from direct Bun FFI against a sibling
libchdb.soto the publishedchdbnpm package (chdb@3.3.0). The public Maple behavior stays the same:maple start, local ingest/query, checkpoint/restore, and bundled release installs continue to work without requiring users to install Node, npm, ClickHouse, or chDB manually.Key points:
chdbnpmSession.maple + node_modules/chdbinstead ofmaple + libchdb.so.Closes #789
Verification
bun --filter @maple/cli typecheckbun test --max-concurrency=1 apps/cli/test/chdb-args.test.ts apps/cli/test/chdb-timezone.test.ts apps/cli/test/update.test.tsbun --filter @maple/cli test— 539/539 passingOUT_DIR=/private/tmp/maple-chdb-npm-bundle-01a07dd0 scripts/build-local-binary.shmaple, queried/healthand/local/query, confirmedtimezone() = UTCandversion() = 26.7.2.1Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.