markap14 commented on pull request #4780: URL: https://github.com/apache/nifi/pull/4780#issuecomment-843567771
Thank you @JonathanKessler! This is a great improvement. I can also see a future improvement that would be great where we could use this information in the Status page in order to sort queues by the Average Queue Time, etc. so that it will help to determine where bottlenecks may be in the flow, etc. The only issue I had with the update was the verbiage used in the ConnectionStatusDescriptor (these stats are not related to the last 5 minutes). So I tweaked the descriptions there, changed the type of Max Queued Duration to DURATION instead of COUNT. Also added in the Average in addition to the max, so that it's provided in the NiFi UI (this won't be shown in the Connection Status, etc. only in the UI). Otherwise, all looks really good. This was a very non-trivial improvement, and I appreciate you sticking with us to get everything sorted. Great contribution, thanks again! -- 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
