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

fix(prover): increase DB polling interval for witness vector generators #697

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

RomanBrodetski
Copy link
Collaborator

Prover DB is under heavy load.

The query that witness vectors generators run to get a new prover job is slow. In theory we shouldn't be running any witness vectors gens for circuits that don't have any jobs queued, but query insights suggest otherwise.

This increases the polling interval - so that even if there are no jobs, we don't perform the query too often

@RomanBrodetski RomanBrodetski changed the title increase DB polling interval for witness vector generators fix(prover): increase DB polling interval for witness vector generators Dec 17, 2023
@RomanBrodetski RomanBrodetski added this pull request to the merge queue Dec 22, 2023
Merged via the queue into main with commit 94579cc Dec 22, 2023
21 of 22 checks passed
@RomanBrodetski RomanBrodetski deleted the rb-increase-polling-interval-vector-gens branch December 22, 2023 10:11
github-merge-queue bot pushed a commit that referenced this pull request Jan 5, 2024
🤖 I have created a release *beep* *boop*
---


##
[10.1.0](prover-v10.0.2...prover-v10.1.0)
(2024-01-05)


### Features

* **prover:** Remove circuit-synthesizer
([#801](#801))
([1426b1b](1426b1b))
* **prover:** Remove old prover
([#810](#810))
([8be1925](8be1925))


### Bug Fixes

* **prover:** increase DB polling interval for witness vector generators
([#697](#697))
([94579cc](94579cc))
* **prover:** Remove prover-utils from core
([#819](#819))
([2ceb911](2ceb911))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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