-
Notifications
You must be signed in to change notification settings - Fork 56
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
wip: proposer #399
base: main
Are you sure you want to change the base?
wip: proposer #399
Conversation
|
…roof_request_time
b727ebb
to
836a666
Compare
…hunk batch insertion requests to avoid pg overload
proposer/succinct/src/proposer.rs
Outdated
let requests = self | ||
.driver_config | ||
.driver_db_client | ||
.fetch_requests_with_status( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This query loads all of the proof bytes for every request which could be slow / use a lot of memory. Could either not include that column or change this query to just return counts
proposer/succinct/src/proposer.rs
Outdated
let completed_range_proofs = self | ||
.driver_config | ||
.driver_db_client | ||
.fetch_completed_range_proofs( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this query doesn't need to return proof bytes?
No description provided.