icefury71 commented on pull request #6890: URL: https://github.com/apache/incubator-pinot/pull/6890#issuecomment-843606952
> @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. > > We can certainly do it in steps. Please let me know your thoughts. Sounds good +1 on this -- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
