Zeke Foppa
Zeke Foppa
This is a breaking-ish change (because users may have used a word that we might end up deciding to reserve).
Let's also enforce this in the rust macro system. (It's already done in C#)
Thanks for the report! @joshua-spacetime is on it 🏃
Adding this to 1.0 release since it was created recently.
Thank you for filing this and iterating on the design+internals a bit! I've triaged this. We'll discuss more as a team and come up with a path forward.
> ``` > tenari@dzs-MacBook-Pro SpacetimeDB % cargo run --bin spacetimedb-update -- --root-dir ~/ > Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.20s > Running `target/debug/spacetimedb-update --root-dir /Users/tenari/` > error:...
This looks like it's due to the fact that we're parsing args "manually" in `crates/update/src/proxy.rs` (specifically, in `PartialCliArgs::parse`).
We could consider putting this in our "testing & error handling" release.
This only happens in the interactive view. The queries succeed without complaint when they're provided on the command line.
This seems to be due to passing `with_stats: true` to `run_sql`. If I change it to `false`, we do not see this issue.