Skip to content

proxyd: Add batch size metric and configurable max#3545

Merged
mslipper merged 4 commits intodevelopfrom
feat/proxyd-batch-metrics
Sep 23, 2022
Merged

proxyd: Add batch size metric and configurable max#3545
mslipper merged 4 commits intodevelopfrom
feat/proxyd-batch-metrics

Conversation

@mslipper
Copy link
Contributor

@mslipper mslipper commented Sep 23, 2022

The max batch size will be overwritten if it is over MaxBatchRPCCallsHardLimit.

Builds on #3544.

@changeset-bot
Copy link

changeset-bot bot commented Sep 23, 2022

🦋 Changeset detected

Latest commit: 0ba0b84

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/proxyd Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mslipper mslipper force-pushed the feat/proxyd-batch-metrics branch from 20a3033 to c551bba Compare September 23, 2022 20:07
@mslipper mslipper requested review from Inphi and tynes September 23, 2022 20:07
The max batch size will be overwritten if it is over `MaxBatchRPCCallsHardLimit`.

Builds on #3544.
@mergify
Copy link
Contributor

mergify bot commented Sep 23, 2022

Hey @mslipper! This PR has merge conflicts. Please fix them before continuing review.

@mergify mergify bot added the conflict label Sep 23, 2022
@mslipper mslipper force-pushed the feat/proxyd-batch-metrics branch from 265169c to 55fc295 Compare September 23, 2022 20:21
@mergify mergify bot removed the conflict label Sep 23, 2022
@mslipper mslipper merged commit 7121648 into develop Sep 23, 2022
@mslipper mslipper deleted the feat/proxyd-batch-metrics branch September 23, 2022 21:06
This was referenced Sep 29, 2022
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.

2 participants