ankitsultana opened a new pull request, #14286:
URL: https://github.com/apache/pinot/pull/14286

   Defines `TimeSeries#getId()` semantics and semantics for the `__name__` 
label value in the broker response.
   
   There aren't any major changes here; we are just merely defining
   
   tl;dr
   
   ### ID Semantics
   
   * `String TimeSeries#_id` is the unique identifier for a series in an 
execution context, and should be used to uniquely identify series.
   * To accommodate scenarios where unions are needed, we already use 
`Map<Long, List<TimeSeries>>` in series blocks.
   * Developers should use a hash algorithm to map the Series String ID to a 
Long, so it could be used in series-block maps.
   
   ### name Tag Semantics in Broker Response
   
   Prometheus returns a meta label `__name__` in its response. By default we 
use a serialized form of tag key values, but languages can overwrite this by 
adding their own tag.
   
   ### Bug Fixes
   
   Additionally I have also fixed the TimeBuckets#getRangeSeconds.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to