Github user bgaff commented on the pull request:
https://github.com/apache/trafficserver/pull/663#issuecomment-221442464
Fwiw I think it might make sense to fully remove event_, nothing really
needs it. It just helps when debugging if you couldn't get a handle to a
request/response object you can determine what state the SM was in when you
attempted to get it to identify user error, other than that it's not used
anywhere, right?
One thing I was thinking was that it might make sense to remove all that
handle caching stuff and initializing code and just fetch the handles in
Transaction::get(Client/Server)(Request/Response)(), thoughts?
Other than that I'm ð with this change.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
