Skip to content

Conversation

@jait91
Copy link
Contributor

@jait91 jait91 commented Nov 19, 2025

  • refactor perf test
  • remove unnecessary round timer from round managers
  • add finalized flag to blocks and add recovery mechanism on startup (retry storing commitments to DB and finalize the block)
  • fatal exit on processing errors - exit immediately on round processing failures, recovery handles on restart

@jait91 jait91 requested a review from MastaP November 19, 2025 06:27
@MastaP MastaP requested a review from lploom November 19, 2025 18:08
Copy link
Contributor

@lploom lploom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MastaP MastaP added this to Unicity Nov 28, 2025
@MastaP MastaP moved this to Test in Unicity Nov 28, 2025
@b3y0urs3lf
Copy link
Contributor

b3y0urs3lf commented Dec 15, 2025

Tested locally using GO aggregator commit b271ce2.

  1. It is possible to start aggregator in sharding mode (2 shards), and successfully submit commitments to it (commit + get inclusion proof).
  2. It is possible to start aggregator in sharding HA mode (2 shards), and successfully submit commitments to it (commit + get inclusion proof).
image
  1. Tested that it is possible to execute perf test on top of sharding in HA mode
image 4. Also checked that after droping mongoDB container related to specific sharding node, then this node would crash and when mongoDb container would be UI it would be able to establish connection to it (tested in sharding mode without HA). image image 5. ALso I was able to sucessfuly add sharding config to subscription sevice image And I was able to send successfully commitments through subscription service.

Merging and moving task to done.

@b3y0urs3lf b3y0urs3lf merged commit d858409 into main Dec 15, 2025
2 checks passed
@b3y0urs3lf b3y0urs3lf deleted the sharding-improvements branch December 15, 2025 14:30
@github-project-automation github-project-automation bot moved this from Test to Done in Unicity Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants