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

feat(ics-033): optimize client updates in multi-hop protocol #1213

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

VolodymyrBg
Copy link

Implemented an optimization in the multi-hop protocol to update clients with specific missing block heights rather than always using the latest block height. This change reduces the likelihood of needing additional client updates on subsequent chains in the multi-hop path, improving overall efficiency.
The implementation:

  1. Added a new UpdateClientToHeight function to the Chain struct
  2. Modified the client update logic to use the specific consensus height when available
  3. Falls back to the existing behavior of updating to the latest height when no specific height is provided
    This change addresses the TODO comment in the multi-hop specification while maintaining backward compatibility with existing code.

@Josealbet
Copy link

Cómo unir verificación

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.

2 participants