Home GnuPG

After a fulltext write to a particular service fails, keep trying writes to…
304d19f92a7bUnpublished

Unpublished Commit · Learn More

Repository Importing: This repository is still importing.

Description

After a fulltext write to a particular service fails, keep trying writes to other services

Summary:
Ref T12450. Currently, if a write fails, we stop and don't try to write to other index services. There's no technical reason not to keep trying writes, it makes some testing easier, and it would improve behavior in a scenario where engines are configured as "primary" and "backup" and the primary service is having some issues.

Also, make "no writable services are configured" acceptable, rather than an error. This state is probably goofy but if we want to detect it I think it should probably be a config-validation issue, not a write-time check. I also think it's not totally unreasonable to want to just turn off all writes for a while (maybe to reduce load while you're doing a background update).

Test Plan:

  • Configured a bad ElasticSearch engine and a good MySQL engine.
  • Ran bin/search index ... --force.
  • Saw MySQL get updated even though ElasticSearch failed.

Reviewers: chad, 20after4

Reviewed By: 20after4

Maniphest Tasks: T12450

Differential Revision: https://secure.phabricator.com/D17599

Details

Provenance
epriestley <git@epriestley.com>Authored on Apr 2 2017, 6:53 PM
Parents
rPHAB0f144d29e920: When "cluster.search" changes, don't trust the old index versions
Branches
Unknown
Tags
Unknown

Event Timeline