shinrich commented on pull request #8281: URL: https://github.com/apache/trafficserver/pull/8281#issuecomment-907264284
The diagram is in here https://docs.google.com/document/d/1sK6AyR-1K795NHsJqpRp4pjZRJ7T5KM78uJlRFMpKiI/edit#heading=h.u0corle7ojrm (second page). The problem is that we have two dimensions of division. Http1/Http2 and client/server. So we already have PoolableSession in the ProxySession hierarchy to deal with the client/server division. This split addresses both the Http1 and Http2 versions of the Server classes. Once we put that into the ProxySession Hierarchy, there isn't a place to pull in the Http2CommonSession logic without also affecting the Http1 classes. At least I couldn't see one. So ended up treating it as a mix-in just for the Http2 classes. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
