-
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: update milvus.yaml #31832
enhance: update milvus.yaml #31832
Conversation
Invalid PR Title Format Detected Your PR submission does not adhere to our required standards. To ensure clarity and consistency, please meet the following criteria:
Required Title Structure:
Where Example:
Please review and update your PR to comply with these guidelines. |
@tedxu E2e jenkins job failed, comment |
1 similar comment
@tedxu E2e jenkins job failed, comment |
6351ea6
to
88388bc
Compare
@tedxu E2e jenkins job failed, comment |
88388bc
to
22ad3c8
Compare
@tedxu E2e jenkins job failed, comment |
22ad3c8
to
4f1d8c4
Compare
@tedxu Please associate the related issue to the body of your Pull Request. (eg. “issue: #”) |
@tedxu E2e jenkins job failed, comment |
/lgtm |
4f1d8c4
to
5a157df
Compare
@tedxu E2e jenkins job failed, comment |
1 similar comment
@tedxu E2e jenkins job failed, comment |
@tedxu ut workflow job failed, comment |
@tedxu E2e jenkins job failed, comment |
/run-cpu-e2e |
@tedxu E2e jenkins job failed, comment |
1 similar comment
@tedxu E2e jenkins job failed, comment |
/run-cpu-e2e |
@tedxu E2e jenkins job failed, comment |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #31832 +/- ##
==========================================
+ Coverage 81.68% 81.73% +0.04%
==========================================
Files 999 999
Lines 123915 123915
==========================================
+ Hits 101218 101277 +59
+ Misses 18819 18751 -68
- Partials 3878 3887 +9
|
5be2c60
to
ef8600f
Compare
|
||
# Related configuration of queryCoord, used to manage topology and load balancing for the query nodes, and handoff from growing segments to sealed segments. | ||
queryCoord: | ||
taskMergeCap: 1 |
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.
Any description or comments here?
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.
Will fill the descriptions in future PRs, thanks!
Signed-off-by: Ted Xu <ted.xu@zilliz.com>
Signed-off-by: Ted Xu <ted.xu@zilliz.com>
Signed-off-by: Ted Xu <ted.xu@zilliz.com>
Signed-off-by: Ted Xu <ted.xu@zilliz.com>
Signed-off-by: Ted Xu <ted.xu@zilliz.com>
ef8600f
to
53bf2c5
Compare
@tedxu ut workflow job failed, comment |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: congqixia, tedxu 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 |
server: # server side configuration for natsmq. | ||
port: 4222 # 4222 by default, Port for nats server listening. | ||
storeDir: /var/lib/milvus/nats # /var/lib/milvus/nats by default, directory to use for JetStream storage of nats. | ||
maxFileStore: 17179869184 # (B) 16GB by default, Maximum size of the 'file' storage. |
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.
Why we choose to delete default value&unit in these params?
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.
OK. This is generated by code, code needs to add units&default value in future
See #32168