Skip to content

fix(coprocessor): persist computed ct128 as local backup#1757

Closed
goshawk-3 wants to merge 1 commit intorelease/0.10.xfrom
georgi/sns-worker/ciphertext128-table
Closed

fix(coprocessor): persist computed ct128 as local backup#1757
goshawk-3 wants to merge 1 commit intorelease/0.10.xfrom
georgi/sns-worker/ciphertext128-table

Conversation

@goshawk-3
Copy link
Copy Markdown
Contributor

@goshawk-3 goshawk-3 commented Jan 9, 2026

Persist computed ct128 values in a dedicated ciphertexts128 table as a failover mechanism when S3 uploads fail and require retry.

Entries are deleted after a successful L2 add_ciphertext call. As an additional safeguard, a periodic GC procedure removes stale entries in cases where the txn-sender fails to clean up as expected.

fixes: https://github.com/zama-ai/fhevm-internal/issues/761
fixes: https://github.com/zama-ai/fhevm-internal/issues/865

@cla-bot cla-bot bot added the cla-signed label Jan 9, 2026
@goshawk-3 goshawk-3 force-pushed the georgi/sns-worker/ciphertext128-table branch from 7041809 to a299426 Compare January 9, 2026 08:19
@mergify
Copy link
Copy Markdown

mergify bot commented Jan 9, 2026

🧪 CI Insights

Here's what we observed from your CI run for 4acec54.

🟢 All jobs passed!

But CI Insights is watching 👀

@goshawk-3 goshawk-3 force-pushed the georgi/sns-worker/ciphertext128-table branch from a299426 to c8fd170 Compare January 9, 2026 08:24
Persist computed ct128 values in a dedicated `ciphertexts128` table as a
failover mechanism when S3 uploads fail and require retry.

Entries are deleted after a successful L2 `add_ciphertext` call. As an
additional safeguard, a periodic GC procedure removes stale entries in
cases where the txn-sender fails to clean up as expected.
@goshawk-3 goshawk-3 force-pushed the georgi/sns-worker/ciphertext128-table branch from c8fd170 to 4acec54 Compare January 9, 2026 14:18
@goshawk-3 goshawk-3 marked this pull request as ready for review January 9, 2026 14:39
@goshawk-3 goshawk-3 requested a review from a team as a code owner January 9, 2026 14:39
antoniupop
antoniupop previously approved these changes Jan 9, 2026
@goshawk-3 goshawk-3 changed the base branch from release/0.10.x to main January 12, 2026 08:25
@goshawk-3 goshawk-3 dismissed antoniupop’s stale review January 12, 2026 08:25

The base branch was changed.

@goshawk-3 goshawk-3 requested review from a team and dartdart26 as code owners January 12, 2026 08:25
@goshawk-3 goshawk-3 changed the base branch from main to release/0.10.x January 12, 2026 08:26
@dartdart26
Copy link
Copy Markdown
Collaborator

@goshawk-3 Is this still relevant?

@goshawk-3
Copy link
Copy Markdown
Contributor Author

@goshawk-3 Is this still relevant?

It is no longer relevant, as it has been merged into main ref: #1790.

@goshawk-3 goshawk-3 closed this Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants