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.

Sign up for GitHub

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

Jump to bottom

Attestation Users SBAT use the new precompile #1169

Closed
nambrot opened this issue Oct 1, 2019 · 0 comments · Fixed by #1248
Closed

Attestation Users SBAT use the new precompile #1169

nambrot opened this issue Oct 1, 2019 · 0 comments · Fixed by #1248
Assignees
@m-chrzan
Labels
enhancement an improvement to an existing feature Priority: P2 Major protocol All issues relating to protocol packages triaged label for issues that have been assigned a priority

Comments

@nambrot
Copy link
Contributor

nambrot commented Oct 1, 2019

Expected Behavior

Use the new precompile

Current Behavior

Currently use Validators#getValidators which is very expensive
@m-chrzan m-chrzan self-assigned this Oct 1, 2019
@nambrot nambrot added the protocol All issues relating to protocol packages label Oct 2, 2019
@m-chrzan m-chrzan mentioned this issue Oct 8, 2019
Merged
@nityas nityas added enhancement an improvement to an existing feature Priority: P2 Major triaged label for issues that have been assigned a priority labels Oct 15, 2019
This was referenced Jan 17, 2022
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@m-chrzan m-chrzan

Labels
enhancement an improvement to an existing feature Priority: P2 Major protocol All issues relating to protocol packages triaged label for issues that have been assigned a priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use validator set precompiles in Attestations celo-org/celo-monorepo
3 participants
@nambrot @nityas @m-chrzan