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

Pass bucket list size from meta to simulation NetworkConfig. #129

Closed
wants to merge 3 commits into from

Conversation

dmkozh
Copy link
Contributor

@dmkozh dmkozh commented Apr 9, 2024

What

Pass bucket list size from meta to NetworkConfig.

Why

Unfortunately, Core currently doesn't emit the bucket list size config entry changes in the meta, so the only way to fetch the proper value as of now is to use the meta field directly.

Known limitations

N/A

Unfortunately, Core currently doesn't emit the bucket list size config entry changes in the meta, so the only way to fetch the proper value as of now is to use the meta field directly.
@2opremio
Copy link
Contributor

Are the updated dependencies based off protocol 20 or 21? (e.g. XDR-wise)

@dmkozh
Copy link
Contributor Author

dmkozh commented Apr 11, 2024

Are the updated dependencies based off protocol 20 or 21? (e.g. XDR-wise)

The dependencies here were protocol 20 (the bump was necessary due to meta extension changes). But since we have decided to only include the fix with v21 env release, the deps in this PR don't make much sense. So I guess you could patch the code (but not dependency) changes from this PR into the PR that would do the 20->21 transition.

@dmkozh dmkozh closed this Apr 16, 2024
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.

3 participants