Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

Eqs remove unbounded queries #1191

Merged
merged 4 commits into from
Jul 12, 2022
Merged

Eqs remove unbounded queries #1191

merged 4 commits into from
Jul 12, 2022

Conversation

sveitser
Copy link
Contributor

I have issues running the tests locally

Close #1190.

@sveitser
Copy link
Contributor Author

The slither error is strange and not related to this PR. The rev we fetch does exist. I don't know why fetchGit thinks it does not exist.

Instead of resetting the backoff to the minimum, reduce it by a factor
of 2 after succesfully fetching new events.
@philippecamacho
Copy link
Collaborator

All the tests pass on my machine @sveitser.

Copy link
Collaborator

@philippecamacho philippecamacho left a comment

Choose a reason for hiding this comment

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

LGTM

This reverts commit 88794ce.

Let's go with the change that we all agree on first.
@sveitser sveitser enabled auto-merge (squash) July 12, 2022 13:31
@sveitser sveitser merged commit e6b00dd into main Jul 12, 2022
@sveitser sveitser deleted the eqs-remove-unbounded-queries branch July 12, 2022 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EQS: unbounded queries can lead to unnecessary load
2 participants