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/11/20 22:21:25 UTC

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4842: [TE] add detection health for the alerts list

jihaozh opened a new pull request #4842: [TE] add detection health for the alerts list
URL: https://github.com/apache/incubator-pinot/pull/4842
 
 
   - Add detection health in the alert list endpoint.
   - Store the detection health in detection config DTO and periodically update it in the monitoring tasks. So that when a user opens the page, it will avoid the time to generate the detection health.
   - Add a cache for loading the detection list, subscription group list, and application list so that it won't need to fetch all the metadata from the database when the user opens the page. The cache will be warmed up when the application starts and after that will be updated periodically. This reduces the alert list page opening latency from around 3s to around 0.3s in my testing. 
   - Minor change to the frontend to move away from the legacy endpoints.

----------------------------------------------------------------
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