-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
enhance: add configurable memory index load predict memory usage factor #30561
enhance: add configurable memory index load predict memory usage factor #30561
Conversation
\hold, wait for related pr: #30473 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #30561 +/- ##
==========================================
- Coverage 80.75% 80.46% -0.29%
==========================================
Files 964 964
Lines 137205 137267 +62
==========================================
- Hits 110799 110458 -341
- Misses 22696 23084 +388
- Partials 3710 3725 +15
|
2b0bbeb
to
9ef83f1
Compare
9ef83f1
to
3edbc8b
Compare
rerun ut |
3 similar comments
rerun ut |
rerun ut |
rerun ut |
3edbc8b
to
2b4eff6
Compare
@chyezh ut workflow job failed, comment |
rerun ut |
1 similar comment
rerun ut |
2b4eff6
to
a898ea2
Compare
@@ -76,7 +76,7 @@ func DefaultParams() map[string]string { | |||
params.LocalStorageCfg.Path.Key: path.Join("/tmp", testPath), | |||
params.CommonCfg.StorageType.Key: "local", | |||
params.DataNodeCfg.MemoryForceSyncEnable.Key: "false", // local execution will print too many logs | |||
params.CommonCfg.GracefulStopTimeout.Key: "10", | |||
params.CommonCfg.GracefulStopTimeout.Key: "30", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too short query node graceful timeout.
Causing querynode_test's stop testing failure.
(search will return service not ready).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only in integration test
@chyezh E2e jenkins job failed, comment |
@chyezh E2e jenkins job failed, comment |
554163a
to
232d423
Compare
@chyezh E2e jenkins job failed, comment |
/run-cpu-e2e |
1 similar comment
/run-cpu-e2e |
232d423
to
ab277b1
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chyezh, czs007 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@chyezh E2e jenkins job failed, comment |
/run-cpu-e2e |
3 similar comments
/run-cpu-e2e |
/run-cpu-e2e |
/run-cpu-e2e |
Signed-off-by: chyezh <chyezh@outlook.com>
ab277b1
to
6a72751
Compare
/lgtm |
…or (milvus-io#30561) related pr: milvus-io#30475 Signed-off-by: chyezh <chyezh@outlook.com>
related pr: #30475