remove the assert on AnalyzerWrapper
see https://issues.apache.org/jira/browse/LUCENE-5170
Removed todo
Added some comments regarding the acknowledgement logic in MetaDataMappingService.putMapping

Made left over cluster state debug log entry less verbose.
lazily create the no shard available exception
Make RestSearchAction#parseSearchXXX(RestRequest) public
When building a plugin with a new search endpoint, you need to parse the request as a searchRequest.

Methods exist in RestSearchAction class but are private.

We will modify them to be public static. This applies to:

* `RestSearchAction#parseSearchRequest(RestRequest)`
* `RestSearchAction#parseSearchSource(RestRequest)`

 Closes #3499.
Allow to update the _source mapping exclude/include dynamically when we merge mappings.

Closes #3491
UpdateMappingTests -  updateDefaultMappingSettings now creates the index with a mapping.
function score test cleanup
- also, properly report on the failed assertion in toFloat
- use function score in the explain compared to custom score
- use the Tests suffix convention
Raise default DeleteIndex Timeout

Currently the timeout for an delete index operation is set to 10 seconds.
Yet, if a full flush is running while we delete and index this can
easily exceed 10 seconds. The timeout is not dramatic ie. the index
will be deleted eventually but the client request is not acked which
can cause confusion. We should raise it to prevent unnecessary confusion
especially in client tests where this can happen if the machine is pretty busy.

The new timeout is set to 60 seconds.

Closes #3498
Fix test include pattern to include *Test.class

We missed *Test.class which is not our convention but we could miss
some tests. We should better include the *Test.class tests as well.
Fix assert to check the deviation rather than the absolute difference.

Deviation should be less or equal to 0.01 ~ 1% after the cast.
empty numeric field data should retain the correct num docs
the fact that there are no values in the numeric field data, doesn't mean there are no docs, behavior should be to the bytes variant
rename class to conform with Tests suffix
no need doc action test to check count in before class
 - also, since we randomize client transports, no need for specific classes to test for it, we test different clients across all our tests
empty double/long values should return 0
to conform with all other implementations (non empty), they getValue when there is no value associated with a doc should be 0