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

Update Solr replication factor #620

Merged
merged 4 commits into from
May 1, 2024
Merged

Update Solr replication factor #620

merged 4 commits into from
May 1, 2024

Conversation

hectorcorrea
Copy link
Member

@hectorcorrea hectorcorrea commented May 1, 2024

Closes #584

Below is how the new Solr collection looks in staging with replication factor = 3

Screenshot 2024-05-01 at 1 48 14 PM

@hectorcorrea hectorcorrea marked this pull request as draft May 1, 2024 17:35
@@ -110,7 +110,7 @@ def self.create_collection(solr_alias_uri, collection_name)
create_query = build_uri(
base_uri: solr_alias_uri,
path: "/solr/admin/collections",
query: "action=CREATE&name=#{collection_name}&collection.configName=#{config_set}&numShards=1&replicationFactor=2"
query: "action=CREATE&name=#{collection_name}&collection.configName=#{config_set}&numShards=1&replicationFactor=3"
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the real change on this PR

@hectorcorrea hectorcorrea marked this pull request as ready for review May 1, 2024 17:54
Copy link
Contributor

@bess bess left a comment

Choose a reason for hiding this comment

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

Thank you!

@@ -26,6 +26,11 @@ namespace :index do
DspaceResearchDataHarvester.harvest(false)
SolrCloudHelper.collection_writer_commit!
Rails.logger.info "Indexing: Harvesting and indexing DataSpace research data collections completed"
rescue => ex
# DataSpace errors quite often.
# Log the exception and let other process continue.
Copy link
Contributor

Choose a reason for hiding this comment

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

🎉

@bess bess merged commit 4594a1a into main May 1, 2024
5 checks passed
@bess bess deleted the 584-replication branch May 1, 2024 18:09
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.

Replication factor for Solr in production and staging
3 participants