Conversation
70ea73b to
d0d0d24
Compare
d0d0d24 to
27fc271
Compare
62bbd3e to
098476f
Compare
joshua-spacetime
left a comment
There was a problem hiding this comment.
Looks great! We also need to be able to build queries over views, but otherwise its ready to merge.
54e6ebb to
76375c0
Compare
gefjon
left a comment
There was a problem hiding this comment.
Can you update the Rust sdk-test clients to use the query builder rather than raw SQL strings?
Co-authored-by: joshua-spacetime <josh@clockworklabs.io> Signed-off-by: Shubham Mishra <shivam828787@gmail.com>
I don't mind if this is done in a different patch. |
|
/update-llm-benchmark |
LLM Benchmark Results (ci-quickfix)
Compared against master branch baseline Generated at: 2026-01-21T11:35:14.893Z Failure Analysis (click to expand)Benchmark Failure AnalysisGenerated from: Summary
SpacetimeDB Benchmark Test Failure AnalysisRust / rustdoc_json FailuresCompile/Publish Errors (3 failures)t_002_scheduled_table
t_015_product_type_columns
Other Failures (5 failures)t_003_struct_in_table
C# / docs Failures (4 total)Other Failures (4 failures)t_014_elementary_columns
Summary of Recommendations
|
|
/update-llm-benchmark |
1fb52b7 to
b578da1
Compare
Description of Changes
Client Query builder for rust, as per proposal - https://github.com/clockworklabs/SpacetimeDBPrivate/pull/2356.
TypedSubscriptionBuilderinsdks/rustas mentioned in proposalAPI and ABI breaking changes
NA, additive changes.
Expected complexity level and risk
2
Testing
Added a test.