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

Modify Bridge Params #800

Merged
merged 2 commits into from
Mar 28, 2022
Merged

Modify Bridge Params #800

merged 2 commits into from
Mar 28, 2022

Conversation

temaniarpit27
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Mar 24, 2022

POS-272 Modify Bridge Params 

We also need to fix this params —
https://github.com/maticnetwork/heimdall/blob/master/bridge/setu/util/common.go#L57-L58

Currently, task delay is only 6 seconds and retry is at 12 seconds. This could also increase retry frequency if state sync is not processed in 6 seconds which is bad
TaskDelayBetweenEachVal = 6 * time.Second
RetryTaskDelay          = 12 * time.Second

@temaniarpit27 temaniarpit27 marked this pull request as ready for review March 24, 2022 10:10
@ssandeep ssandeep merged commit 7d44d45 into master Mar 28, 2022
@vcastellm vcastellm deleted the arpit/pos-272 branch March 28, 2022 17:29
0xKrishna pushed a commit that referenced this pull request Apr 5, 2022
@temaniarpit27 @0xKrishna
Modify Bridge Params (#800)
bc8703a 
* change delay to 10 sec

* Update common.go
manav2401 pushed a commit that referenced this pull request Apr 7, 2022
@temaniarpit27 @manav2401
Modify Bridge Params (#800)
0efee9d 
* change delay to 10 sec

* Update common.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ssandeep ssandeep ssandeep approved these changes

@0xKrishna 0xKrishna 0xKrishna approved these changes

@jdkanani jdkanani Awaiting requested review from jdkanani

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@temaniarpit27 @ssandeep @0xKrishna