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

SNO-124-make-elasticsearch-shards-configurable #295

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Bek
Copy link
Collaborator

@Bek Bek commented Oct 5, 2020

No description provided.

@Bek Bek changed the title configurable shards SNO-124 configurable shards Oct 5, 2020
@keenangraham keenangraham changed the title SNO-124 configurable shards SNO-124-make-elasticsearch-shards-configurable Oct 7, 2020
@Bek Bek force-pushed the SNO-124-deploy-shard-settings branch from d545033 to f8dcb87 Compare October 20, 2020 16:11
Comment on lines +8 to +9
elasticsearch.shards.primary = 1
elasticsearch.shards.replicate = 0
Copy link
Member

Choose a reason for hiding this comment

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

@keenangraham @Bek I think we should start with adding the ability to configure the shards without actually changing it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

With version 5, we don’t have the total shards limit and we can leave the default number as before. We should still change it to minimum for single machine demos, what do you think? I will look to it.

Copy link
Member

Choose a reason for hiding this comment

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

I don't think it matters much on a single-node demo; they already index just as fast and they have a low search load. What difference do you think it would make?

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