jetty/jetty.project

Improve ConnectionStatistics to report per-protocol information

Open

#4.184 aberto em 11 de out. de 2019

Ver no GitHub
 (10 comments) (0 reactions) (2 assignees)Java (1.913 forks)batch import
EnhancementHelp WantedLow PriorityPinned

Métricas do repositório

Stars
 (3.701 stars)
Métricas de merge de PR
 (Mesclagem média 6d 14h) (48 fundiu PRs em 30d)

Description

Currently, ConnectionStatistics listens to the "connection closed" event, and when it happens it retrieves the connection information from the Connection object passed in the event.

Would be great if this information could be split per-protocol.

This would involve:

  1. a new Connection.getProtocol() method so that it would be possible to know what protocol was that connection speaking
  2. a different implementation of ConnectionStatistics to store per-protocol information (as well as the totals)
  3. Possibly a ConnectionStatisticsMBean that displays the per-protocol data in a JMX friendly way. JMX's CompositeType represents a struct; in this case it can have these fields: [protocol, bytesIn, bytesOut, messagesIn, messagesOut]. JMX's CompositeData is an instance of the struct with the actual values for those fields. JMX's TabularData is a Map where you can map a String to a CompositeData (I think - this class is so obscure I completely forgotten it was so obscure). Point being that either via an array of CompositeData, or via TabularData we can return the connection information split by protocol.

Guia do colaborador