search

mattbaird / elastigo

A Go (golang) based Elasticsearch client library.
Apache License 2.0
943 stars 241 forks source link

Support json.UseNumber option #262

Open pitr opened 8 years ago

pitr commented 8 years ago

json package unmarshalls numbers in JSON to float64, which is not optimal if you are working with big integers, see this SO question for example.

I'd like to add a setting to call Decoder.UseNumber which converts numbers to Number. Is this something that makes sense to authors of this library?

pitr commented 8 years ago

bump. would love some feedback on this

vrecan commented 8 years ago

I think this would be good to support but I don't have the time to implement. If a PR shows up with tests for it I would merge it! :)