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

doc(pd): add initial-store-count comments in application.yml #2587

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

haohao0103
Copy link
Contributor

close #2579

PD cluster will assess the overall health of the cluster. If the number of store nodes is less than the initial-store-count, the cluster will be unavailable

PD cluster will assess the overall health of the cluster. If the number of store nodes is less than the initial-store-count, the cluster will be unavailable
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. pd PD module labels Jul 15, 2024
Copy link

codecov bot commented Jul 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.48%. Comparing base (03b40a5) to head (2fd4b69).
Report is 2 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (03b40a5) and HEAD (2fd4b69). Click for more details.

HEAD has 5 uploads less than BASE
Flag BASE (03b40a5) HEAD (2fd4b69)
6 1
Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2587       +/-   ##
=============================================
- Coverage     52.49%   39.48%   -13.01%     
- Complexity      586      635       +49     
=============================================
  Files           643      643               
  Lines         53393    53393               
  Branches       7018     7018               
=============================================
- Hits          28026    21081     -6945     
- Misses        22538    30015     +7477     
+ Partials       2829     2297      -532     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@imbajin imbajin changed the title Add comments to the initial-store-count attribute in pd application.yml doc: add attribute comments in pd application.yml Jul 15, 2024
@imbajin imbajin changed the title doc: add attribute comments in pd application.yml doc(pd): add initial-store-count comments in application.yml Jul 15, 2024
@@ -70,7 +70,6 @@ store:
monitor_data_interval: 1 minute
# Retention time of monitoring data is 1 day; day, month, year
monitor_data_retention: 1 day
initial-store-count: 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: remove store.initial-store-count in other similar conf files

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 15, 2024
@imbajin imbajin merged commit 899dc8e into apache:master Jul 16, 2024
14 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer pd PD module size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add comments to the initial-store-count attribute in pd application.yml
3 participants