8000 Revert "Add black logo" by hanxiao · Pull Request #3134 · jina-ai/serve · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Revert "Add black logo" #3134

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 9, 2021
Merged

Revert "Add black logo" #3134

merged 1 commit into from
Aug 9, 2021

Conversation

hanxiao
Copy link
Member
@hanxiao hanxiao commented Aug 9, 2021

Reverts #3133

@hanxiao hanxiao requested a review from a team as a code owner August 9, 2021 20:09
@hanxiao hanxiao merged commit 7394523 into master Aug 9, 2021
@hanxiao hanxiao deleted the revert-3133-add-black-logo branch August 9, 2021 20:09
@github-actions github-actions bot added the size/S label Aug 9, 2021
hanxiao added a commit that referenced this pull request Aug 9, 2021
@hanxiao
Revert "revert add black logo (#3133)" (#3134)"
7901913 
This reverts commit 7394523.
@github-actions github-actions bot added the area/docs This issue/PR affects the docs label Aug 9, 2021
@github-actions
Copy link
github-actions bot commented Aug 9, 2021

Latency summary

Current PR yields:

  • 🐎🐎🐎🐎 index QPS at 1748, delta to last 2 avg.: +22%
  • 😶 query QPS at 36, delta to last 2 avg.: +0%
  • 😶 dam extend QPS at 67136, delta to last 2 avg.: +5%
  • 😶 avg flow time within 1.1683 seconds, delta to last 2 avg.: -32%
  • 🐢🐢 import jina within 0.2911 seconds, delta to last 2 avg.: -21%

Breakdown

Version Index QPS Query QPS DAM Extend QPS Avg Flow Time (s) Import Time (s)
current 1748 36 67136 1.1683 0.2911
2.0.16 1368 33 59496 1.854 0.3591
2.0.15 1475 39 68296 1.5822 0.3829

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@florian-hoenicke florian-hoenicke Awaiting requested review from florian-hoenicke

@Yongxuanzhang Yongxuanzhang Awaiting requested review from Yongxuanzhang

Assignees
No one assigned
Labels
area/docs This issue/PR affects the docs size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hanxiao
0