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

Add election and snapshot configurations as optional parameters #198

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

redouan-rhazouani
Copy link
Contributor

No description provided.

Comment on lines +234 to +239
# RAFT_HEARTBEAT_TIMEOUT: 1
# RAFT_ELECTION_TIMEOUT: 1
# Control how often raft checks if it should perform a snapshot
# RAFT_SNAPSHOT_INTERVAL: 120
# Number of outstanding log entries before performing a snapshot
# RAFT_SNAPSHOT_THRESHOLD: 8192
Copy link
Collaborator

Choose a reason for hiding this comment

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

Red, can you make appropriate comments on each env variable describing what it does and what's the default value?

We have a rule that we try to make descriptive comments on each env variable so that the user would know what a given setting does

@weaviate-git-bot
Copy link

To avoid any confusion in the future about your contribution to Weaviate, we work with a Contributor License Agreement. If you agree, you can simply add a comment to this PR that you agree with the CLA so that we can merge.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

@redouan-rhazouani redouan-rhazouani requested a review from a team as a code owner March 20, 2024 12:02
@jfrancoa jfrancoa force-pushed the raft-configuration branch 2 times, most recently from 969d6c5 to f8a8526 Compare March 21, 2024 16:07
@antas-marcin antas-marcin force-pushed the raft-configuration branch 3 times, most recently from f41f161 to ba8d329 Compare April 19, 2024 19:52
Base automatically changed from raft-configuration to master April 30, 2024 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants