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 2020/07/22 04:26:05 UTC
[GitHub] [incubator-pinot] suvodeep-pyne opened a new pull request #5727: [TE] Added DataSource.getMinDateTime() API implementation for PinotDataSource
suvodeep-pyne opened a new pull request #5727:
URL: https://github.com/apache/incubator-pinot/pull/5727
## Description
The min datetime API is required to fetch the earliest available timestamp
for a given dataset in Pinot.
- By default the API returns -1L for other datasources apart Pinot.
- If the dataset is not present in Pinot, it returns 0
- Else it queries pinot for the timestamp
- `PinotDataSourceMaxTime` has been renamed to `PinotDataSourceTimeQuery`
- The `collectionToPrevMaxDataTimeMap` map has been removed since it was being updated but never queried.
There are NO backward incompatible changes.
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [incubator-pinot] suvodeep-pyne commented on pull request #5727: [TE] Added DataSource.getMinDateTime() API implementation for PinotDataSource
Posted by GitBox <gi...@apache.org>.
suvodeep-pyne commented on pull request #5727:
URL: https://github.com/apache/incubator-pinot/pull/5727#issuecomment-662551757
Thanks @akshayrai
The min date will be useful when trying to figure out how far to look back when figuring out anomalies when a new alert is created.
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [incubator-pinot] akshayrai merged pull request #5727: [TE] Added DataSource.getMinDateTime() API implementation for PinotDataSource
Posted by GitBox <gi...@apache.org>.
akshayrai merged pull request #5727:
URL: https://github.com/apache/incubator-pinot/pull/5727
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org