feat: merge matches topk

[florian-hoenicke]

No description provided.

[codecov]

Codecov Report

Merging #1686 (16ae488) into master (5530733) will increase coverage by 0.57%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1686      +/-   ##
==========================================
+ Coverage   84.20%   84.78%   +0.57%     
==========================================
  Files         132      133       +1     
  Lines        6830     6820      -10     
==========================================
+ Hits         5751     5782      +31     
+ Misses       1079     1038      -41     

Impacted Files	Coverage Δ
jina/flow/base.py	87.01% <0.00%> (-0.56%)	⬇️
jina/parsers/hub/__init__.py	65.00% <0.00%> (-0.12%)	⬇️
jina/docker/hubapi.py
jina/docker/hubapi/local.py	81.70% <0.00%> (ø)
jina/docker/hubapi/remote.py	75.32% <0.00%> (ø)
jina/docker/checker.py	84.90% <0.00%> (+2.76%)	⬆️
jina/docker/hubio.py	70.27% <0.00%> (+7.00%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5530733...16ae488. Read the comment docs.

[JoanFM]

this has to be set arbitrarily and overriden by QueryLangDriver, but setting an env variable is bad since we do not know it will exist. you can write 10

[hanxiao]

the new syntax for env var substituion is https://docs.jina.ai/chapters/yaml/substitute.html#confval-${{ENV.var}}

[JoanFM]

but we should not consider this is there right?

[github-actions]

Latency summary

Current PR yields:

• 😶 index QPS at 1595, delta to last 3 avg.: +1%
• 😶 query QPS at 26, delta to last 3 avg.: -3%

Breakdown

Version	Index QPS	Query QPS
current	1595	26
0.9.13	1551	26
0.9.12	1583	26

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

[JoanFM]

It could be good to add this in one test? maybe take an integration test and change the name and see it works. Plus remove the env var