You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2019/05/30 23:24:07 UTC

[GitHub] [incubator-pinot] jihaozh edited a comment on issue #4262: [TE] add current time series if it's unavailable in predicted baseline endpoint

jihaozh edited a comment on issue #4262: [TE] add current time series if it's unavailable in predicted baseline endpoint
URL: https://github.com/apache/incubator-pinot/pull/4262#issuecomment-497520534
 
 
   > @jihaozh Can you give more details why what will leads to this scenario?
   
   The previous PR added padding so that the graphs will have paddings. https://github.com/apache/incubator-pinot/pull/4260
   
   This PR fixes an edge case that if the current time series is already returned from the baseline provider, it will skip adding it. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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