clockworklabs / clockworklabs/SpacetimeDB
Support calling parameterized views from SQL
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 25.2k
- Forks
- 1.1k
- Avg merge
- 2d 7h
- Merged PRs (30d)
- 46
Description
This issue tracks reimplementing the work from stale PR #3933, which has been closed because it is too far out of date to merge directly.
Original pull request
- PR #3933: Call parameterized views from sql: Execution
- Original author: @mamcx
What the original PR was trying to do
Description of Changes Second of a set PRs for closing of 3489. Stacked against 3791 Expected complexity level and risk: 2 Need to change to passing tx mutable to allow for internal params creation. Testing - [x] Unit test for sql statements - [ ] TODO: For evaluation
Closure context
- This is substantially out of date. I'm going to close this and create an issue.
Reimplementation notes
- Reimplement this work on top of current
masterin a new PR. - Keep the original PR linked as historical context and as a source of useful implementation ideas where still relevant.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the historical PR #3933 and its SQL statement unit tests, then compare that work with the current master branch. Reimplement support for calling parameterized views from SQL and add or update evaluation coverage so the behavior is verified on the current codebase.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust, sql
- Domain
- databases
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 42/100