8000 fix(ci): fixed benchmark CI by maateen · Pull Request #3311 · jina-ai/serve · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(ci): fixed benchmark CI #3311

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 1 commit into from
Aug 31, 2021
Merged

fix(ci): fixed benchmark CI #3311

merged 1 commit into from
Aug 31, 2021

Conversation

maateen
Copy link
Contributor
@maateen maateen commented Aug 31, 2021

Changes:

  • fix(ci): fixed benchmark CI

Signed-off-by: Maksudur Rahman Maateen <maateen@jina.ai>
@maateen maateen requested a review from hanxiao as a code owner August 31, 2021 15:57
@github-actions github-actions bot added size/S area/cicd This issue/PR affects the cicd pipeline area/housekeeping This issue/PR is housekeeping labels Aug 31, 2021
@github-actions
Copy link

Latency summary

Current PR yields:

  • 🐢🐢 index QPS at 1074, delta to last 2 avg.: -13%
  • 🐢🐢 query QPS at 41, delta to last 2 avg.: -22%
  • 🐢🐢 dam extend QPS at 41896, delta to last 2 avg.: -17%
  • 🐢🐢 avg flow time within 1.8071 seconds, delta to last 2 avg.: +26%
  • 🐎🐎🐎🐎 import jina within 0.4426 seconds, delta to last 2 avg.: +22%

Breakdown

Version Index QPS Query QPS DAM Extend QPS Avg Flow Time (s) Import Time (s)
current 1074 41 41896 1.8071 0.4426
2.0.21 1098 45 37489 1.7311 0.3905
2.0.20 1396 60 64001 1.1192 0.3347

Backed by latency-tracking. Further commits will update this comment.

@hanxiao hanxiao merged commit a2e9ebb into master Aug 31, 2021
@hanxiao hanxiao deleted the fix-benchmark-ci branch August 31, 2021 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cicd This issue/PR affects the cicd pipeline area/housekeeping This issue/PR is housekeeping size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0