Commits on Mar 1, 2023

1. api: Extend and improve json-iterator usage …

   For one, this pulls up the histogram-related json-iterator usage from
   prometheus/common into the API client. Previously, the only
   json-iterater usage was here in the API client. But then json-iterator
   was used for the native histogram additions directly in
   prometheus/common, see
   https://github.com/prometheus/common/pull/440/files . This however
   meant that any user of prometheus/common/model would now link in
   json-iterator, even if they are not using the JSON marshaling at
   all. To keep prometheus/common/model more leightweight, this commit
   moves all the json-iterator usage into the API client itself, as it
   was done before for the normal float samples.
   
   This commit also adds an unmarshaling function for native histograms,
   which didn't even exist in prometheus/common/model so far.
   
   It also adds json-iterator marshaling and un-marshaling for
   model.SampleStream, which is only needed for the benchmark
   (BenchmarkSamplesJsonSerialization). This fixes the benchmark such
   that it actually compares json-iterator and std-lib json encoding
   (which didn't work before because the custom marshaling methods of
   model.SampleStream enforced std-lib json encoding for floats and
   json-iterator encoding for histograms in all cases).
   
   I expect this to fix #1179.
   
   Signed-off-by: beorn7 <beorn@grafana.com>

   

   beorn7 committed Mar 1, 2023
   Configuration menu

   • View commit details
   • Copy full SHA for 2236d78
   • Browse repository at this point

   Copy the full SHA

   2236d78 View commit details

   Browse the repository at this point in the history