8000 Add "gateway benchmark-stream"(PRIME-651, PRIME-654, PRIME-655) by vdavid · Pull Request #1138 · sourcegraph/src-cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add "gateway benchmark-stream"(PRIME-651, PRIME-654, PRIME-655) #1138

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

Merged
merged 8 commits into from
Dec 20, 2024

Conversation

vdavid
Copy link
Contributor
@vdavid vdavid commented Dec 16, 2024

This PR:

  • adds the gateway benchmark-stream endpoint similar to gateway benchmark with the capability to test the actual code completion endpoints on SG and CG instances.
  • adds the --use-special-header flag and optional request-level CSV output to the "benchmark gateway" command.

This is done for benchmarking purposes for these three issues:

See those issues for more context.

Test plan

Manual QA: I used the tool several times after the changes.

@camdencheek
Copy link
Member

@vdavid is this ready for review now? And is Code Search who you want reviewing it?

@vdavid vdavid removed the request for review from a team December 16, 2024 19:37
@vdavid
Copy link
Contributor Author
vdavid commented Dec 16, 2024

@camdencheek, very sorry, I have no idea how I got Code Search tagged on this one. Pls ignore.

@camdencheek
Copy link
Member

Haha, no worries. I think we're auto-tagged on this repo since we're codeowners.

@vdavid vdavid changed the title Add "gateway benchmark-stream" Add "gateway benchmark-stream"(PRIME-655) Dec 17, 2024
* Add --use-special-header to gateway benchmark
* Add individual request CSV output to the "benchmark gateway" command
@vdavid
Copy link
Contributor Author
vdavid commented Dec 20, 2024

@slimsag, I've added some more changes to this but it's roughly the same as when you reviewed it, plus I went another quick round on it, adding a bit more docs and the such. I'll merge it to avoid growing it further, but feel free to re-review it and ask for fixes.

@vdavid vdavid merged commit b2e0f1d into main Dec 20, 2024
7 of 8 checks passed
@vdavid vdavid deleted the dv/gateway-benchmark-stream branch December 20, 2024 11:35
@vdavid vdavid changed the title Add "gateway benchmark-stream"(PRIME-655) Add "gateway benchmark-stream"(PRIME-651, PRIME-654, PRIME-655) Dec 20, 2024
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
0