Reduce duplicate tracking of writes in changeset and state
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 25/100
- Issue type
- Refactor
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- cpp
- Domain
- distributed-systems
Research direction
Start with the changeset and current-state handling discussed in PR #4352, focusing on how writes are tracked and how deletes differ. Determine whether retaining only deletes is a valid optimization, and document or implement the resulting change with appropriate validation.
Written by the indexing model from the issue text.
Description
Unless I am missing something, this augments the changeset with all the writes, which mostly duplicate current->state except for deletes. Is that correct? If so, and if this is done for expediency, can we have a ticket that tracks that it's possible to optimise this in the future by adding only deletes here?
Originally posted by @achamayou in https://github.com/microsoft/CCF/pull/4352#discussion_r1014198531
- Dominant language
- C++
- Stars
- 874
- Forks
- 263
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 157
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
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.
More from microsoft/CCF
-
Difficulty 4/5 3-5 days Newbie friendliness 38/100
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
microsoft/CCF#8184 · 1 comment ·
Maintainers usually reply within 1 day
-
Encrypt private ledger data directly into serialised entriesPossibly taken @achamayou claimed this 8 days ago. Opencrypto enhancement performance
Difficulty 5/5 Over a week Newbie friendliness 42/100
microsoft/CCF#8169 · 1 reaction · 2 assignees ·
Maintainers usually reply within 1 day
-
Hybrid (classical crypto + PQ) TLS in CCFMay be free again A pull request for this issue was closed without being merged. Open
Difficulty 5/5 Over a week Newbie friendliness 25/100
Maintainers usually reply within 1 day
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
-
[request] vsg/1.1.16Openupstream update
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
conan-io/conan-center-index#31142 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
FISCO-BCOS/FISCO-BCOS#5684 ·
Maintainers usually reply within 4 days