Coln store/query+rpc - #162
Merged
Merged
Conversation
incipit0
force-pushed
the
coln-store/query+rpc
branch
from
September 12, 2026 14:44
dd46310 to
251e643
Compare
…h a small test of two tx
Just add it, more work to do later on.
Rewrite some of the indexing methods. Introduce TableHandle::inner() which allows internal packages to use PackedRowId methods if they wish to.
Now that they are called TxnLiveRowId.
Also did rename SortedTableSnapshot -> SortedTable. snapshot means something special in coln-store, where it is used to rollback operations.
`TableHandle` handles the packing/unpacking to and from WireRowId <-> PackedRowId. Also fold TableMut into TableHandle, not on TestTable.
An empty commit will return early with a all-zero hash and not modify the commit graph. In the future we might want to introduce an option of something like --allow-empty.
Make txn able to read things from table. For now these are mostly dup code from the StoreHandle. I want to redesign the RW interface from coln-store so that might change. For now, we can read stuff inside a transaction.
Add read/write transactions to coln-store, with typestate. Add convenience methods to do single-shot read/write on the store, internally they will start/finish a transaction for you.
And now it's up to the user to do finalisation of ids! But we don't support query by TxnWireRowId anyway, so should be fine. There is still a `Store::promote` method available which does resolve ids and canoncailise them.
incipit0
force-pushed
the
coln-store/query+rpc
branch
from
September 14, 2026 17:15
251e643 to
4827773
Compare
olynch
marked this pull request as ready for review
September 15, 2026 14:24
Disables coln-query, and build for wasm. We rule derivation and checking by doing that.
incipit0
force-pushed
the
coln-store/query+rpc
branch
from
September 15, 2026 16:49
e70d7c1 to
e2b01fa
Compare
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
coln-query integration + initial model IR + query interface + whatever refactor....