Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(DA): refine DA stream define and implementation #2652

Merged
merged 7 commits into from
Sep 19, 2024

Conversation

popcnt1
Copy link
Collaborator

@popcnt1 popcnt1 commented Sep 19, 2024

Summary

  1. clarify stream, batch, l2 block, chunk ,segment's relations
  2. implement necessary methods of chunk and segment
  3. refine code logic of open-da/celestia submit
  4. refine mainet deploy workflow

Enhanced DA server with chunk and segment handling functionality including compression with lz4. Improved batch submission process, addressing batch data integrity and submission logic.
Introduce batch retrieval and chunk creation from segments. This adds new methods to handle batch data and validate segment consistency, enhancing the chunk and segment interfaces.
Add detailed log messages for segment submission to Celestia and Open-DA nodes. Improves error handling and debug capability by logging segment IDs, namespaces, commitments, and errors.
Refactor deployment workflows for distinct environments. Added separate scripts for testnet and mainnet deployments. Introduced a new constant for default max retry times in DAServerOpenDAActor.
Copy link

vercel bot commented Sep 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rooch-portal ❌ Failed (Inspect) Sep 19, 2024 2:19pm
rooch-portal-v2.1 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 19, 2024 2:19pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
rooch ⬜️ Ignored (Inspect) Visit Preview Sep 19, 2024 2:19pm

Copy link

vercel bot commented Sep 19, 2024

@popcnt1 is attempting to deploy a commit to the Rooch Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@yourmoonlight yourmoonlight left a comment

Choose a reason for hiding this comment

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

LGTM

Updated deploy_rooch_mainnet.sh to exclude the faucet container when stopping and removing Docker containers. This ensures the faucet container stays running during the deployment process.
@popcnt1 popcnt1 merged commit 0c15e33 into rooch-network:main Sep 19, 2024
9 of 10 checks passed
@popcnt1 popcnt1 deleted the refactor/da branch September 19, 2024 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants