Skip to content
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

fix: dont test thread backend in jinad #4148

Merged
merged 1 commit into from
Jan 10, 2022
Merged

Conversation

jacobowitz
Copy link
Contributor

Thread backend for a runtime worker does not work well with containers. Dont test it in jinad

@github-actions github-actions bot added size/S area/testing This issue/PR affects testing labels Jan 10, 2022
@JoanFM JoanFM merged commit 4c6e393 into master Jan 10, 2022
@JoanFM JoanFM deleted the fix-tests-runtime-backend branch January 10, 2022 13:40
@github-actions
Copy link

Latency summary

Current PR yields:

  • 😶 index QPS at 1171, delta to last 2 avg.: +0%
  • 😶 query QPS at 20, delta to last 2 avg.: -5%
  • 😶 dam extend QPS at 35254, delta to last 2 avg.: +7%
  • 😶 avg flow time within 1.5265 seconds, delta to last 2 avg.: -4%
  • 🐢🐢 import jina within 0.4084 seconds, delta to last 2 avg.: -8%

Breakdown

Version Index QPS Query QPS DAM Extend QPS Avg Flow Time (s) Import Time (s)
current 1171 20 35254 1.5265 0.4084
2.6.4 1144 20 31770 1.5918 0.4839
2.6.3 1199 21 33967 1.5934 0.4128

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
Labels
area/testing This issue/PR affects testing size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants