Remove unused variable and fix outdated comment

Change-Id: I0d70d49bbec6232c04ab7d43b0d9eca8cf0405d6
Moving shared code to mediawiki-selenium Ruby gem

Bug: 53579
Change-Id: I0dcc7d7c91ceabb30e9161238592ddacd666deaa
Rename --rebuild to --startOver

--rebuild was confusing because it wasn't the kind of reindex we ever
wanted to run in production.  --startOver carries the right connotations
for what it actually does: it is useful when trying to get the thing
working for the first time but it removes all the data that exists so
shouldn't be used if you don't want a substantial interruption in
service.

Bug: 56164
Change-Id: I6aede782e6a3da5b729d6ea8cd82411fed319837
Add comments about Elasticsearch prod config

We have some opinions on the right way to set up Elasticsearch for
CirrusSearch so we should make those public in the README.

Bug: 56165
Change-Id: I4f960684ad8412bb752d155ae49f7af17afb20c3
Merge "Add comments about Elasticsearch prod config"
Merge "Remove unused variable and fix outdated comment"
Allow boolean operators.

We have escape some of them anyway if they are used wrong, but that
isn't too bad.

Bug: 56026
Change-Id: Iaeedda690b321b1282d9a36505823fa9ab851796
Tests for boolean operators.

Also, update the logging in instructions after switching to the
mediawiki-selenium gem.

Bug: 56026
Change-Id: I1efbb677a679b639b07c3a0ef3e1aca8fe74f6e5
Merge "Allow boolean operators."
Merge "Tests for boolean operators."
Prevent search index updates of nonexistant pages

This also removed textLen as it was deprecated and no longer used.

Bug: 56020
Change-Id: I7ee22d5c1cc9c83c98548a2b9eb97dc3a112be9c
Tests catching some invalid word count causes

I believe this catches all [INVALID] word count causes but I didn't want
the short description to be too presumptious.  This certainly won't count
future potential causes.

Bug: 56020
Change-Id: I6ec21dd804aa7e064b6ea1755377560c00e8af16
Merge "Tests catching some invalid word count causes"
General code cleanup

- Move a couple of classes out of CirrusSearchSearcher since it
  was getting rather long
- Explicitly register all classes if they aren't
- Use anonymous function for prefix setup because I can, remove
  single-function class
- Adjust uses of getIndexName()/getIndex() to go with I1d8aa7f9

Change-Id: I53056c40e72e73205ea1130fa62b4afecd83996c
Don't mention Solr since we don't use it

Change-Id: I7f6a01131334519d9d8a0c3332b507a665263769
Merge "General code cleanup"
Make test more resilient to things taking longer.

Since we're using the the job queue more the update tests have to be a bit
more willing to wait for the changes that they expect to take place.

Bug: 55848
Change-Id: I8099fc37b9f48d907b4252375329cb0905408545
Don't write to _all field.

This cuts the index size by about 10% which should cut memory usage
similarly.

Change-Id: I16240b0900619107e271180406c49055d8124513
Use a job to handle link updates.

The job is immediately executed if it is built in the cli but queued
otherwise.

Bug: 55848
Change-Id: I7bad34fef368a3177c7658d066092c86efec24ad
Merge "Make test more resilient to things taking longer."