Using string formatter for range aggregations. Closes #4432 & #4404

[gauravsinghania]

For an aggregations the field format used depends on the field type. For a number field the x-axis labels are NaN and a numberic formatter would give wrong results, hence forcing a string formatter for a range aggregations. Closes #4432 & #4404

[elasticsearch-release]

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

[gauravsinghania]

@spalger any updates?

[spalger]

Sorry @gauravsinghania I've been aggressively trying to get merged and will get to this soon

#4335

[gauravsinghania]

@spalger no worries, thanks.

[spalger]

Hey @gauravsinghania, I merged master and updated the changes to match the date_range aggregation's implementation. Would you mind taking a look at my pull against your fork and merging it when you get a chance?

[gauravsinghania]

@spalger thanks, I will have a look.

[spalger]

@gauravsinghania the change list looks daunting, but that's because it merges master. If you merge it into your pull request (just hit the green button) then we can move forward with this pull.

[gauravsinghania]

@spalger it was delayed because so far I was using es 1.5.2 and the kibana master doesn't work with it. Is there a work around or I need to use es 2.0. Also if I have to use es 2.0 do i need to get the code or is there any binary available for it (in the repo I can only see executable till 1.7.1) .

[spalger]

@gauravsinghania, thanks so much for taking the initiative here. The more I played with this the more it inspired another approach, which I have taken in #4613. Please take a look at that pr and let me know if it works for you. For now I'm going to close this as the other PR includes these changes.

[gauravsinghania]

@spalger thanks a lot. I will have a look at the other PR.