ReIndexWithCreate improvements #5

Merged
merged 1 commit into from Mar 26, 2013

Projects

None yet

2 participants

@youurayy
  • type now accepts *, which means "all mappings"
  • before the delete check on index document count, the new index is refreshed (it could be paused, or the refresh_interval could be too long)
  • when the target index exists, it is not created or modified, but the reindexing still goes through (this should probably be as an option, as it is potentially dangerous)
  • proper REST response on completion (when an exception happens, this is not yet printed to the REST client (curl) - not sure why)

(sorry for the whitespace changes!)

@karussell
Owner

Sorry, totally missed this here! Nice improvements!

Instead of the explicit and costly refresh call - couldn't we just wait "refresh_interval" seconds?

@karussell karussell merged commit 369afef into karussell:master Mar 26, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment