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 2021/05/18 21:40:29 UTC

[GitHub] [incubator-pinot] mayankshriv commented on pull request #6890: Adding a new Controller API to retrieve ingestion status for realtime…

mayankshriv commented on pull request #6890:
URL: https://github.com/apache/incubator-pinot/pull/6890#issuecomment-843582088


   @icefury71 @npawar
   
   To avoid creating multiple debug api's on servers, I propose to consolidate the one I am creating for segments as follows:
   1. The api consumingSegmentsInfo will be deprecated.
   2. We will create a new api called segmentStatusInfo (or another name such as segmentDebugInfo) which will have:
      - Consumption info from 1.
      - Errors/exceptions related to segment
      - Any other info we want to add in future.
   3. On the controller side, these can all roll up inside the existing table debug endpoint, if we want to further consolidate.
   Please let me know your thoughts.


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