Skip to content

Conversation

ElasticBottle
Copy link
Contributor

@ElasticBottle ElasticBottle commented Aug 8, 2023

Changes

  • Linear
  • Move benchmark script from shell to typescript to enable greater developer ergonomics in post processing
  • No developer lift for benchmark running. Previous iteration required apache benchmark
  • Add time taken for mempool submission

How this PR will be tested

  • pull down winstoni/plat-1058
  • yarn to grab latest packages
  • yarn benchmark against your local server

Output

image

Known Issue

The time taken table currently shows negative time because the createdTimestamp is consistently larger than the submitted or processed timestamp. Likely a docker oddity

@zeet-co
Copy link

zeet-co bot commented Aug 8, 2023

We're building your pull request over on Zeet.
Click me for more info about your build and deployment.
Once built, this branch can be tested at: https://web3-api-akbv-winston-plat-1058.chainsaw-dev.zeet.app before merging 😉

chore(worker): remove fee data info for now bc it's hacky and won't work with non eip1559 evm chains
@socket-security
Copy link

socket-security bot commented Aug 21, 2023

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
autocannon 7.12.0 eval, network, filesystem, shell, environment +24 2.41 MB matteo.collina
@types/autocannon 7.9.1 None +0 20.1 kB types

@farhanW3
Copy link
Contributor

image

@ElasticBottle if you can take a look too 👀 . let me know if you see anything abnormal

@github-actions
Copy link

github-actions bot commented Sep 5, 2023

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

Signed-off-by: farhanW3 <132962163+farhanW3@users.noreply.github.com>
@farhanW3 farhanW3 merged commit be977b7 into main Sep 5, 2023
@farhanW3 farhanW3 deleted the winston/plat-1058 branch September 5, 2023 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants