[
https://issues.apache.org/jira/browse/TS-4481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15299243#comment-15299243
]
ASF GitHub Bot commented on TS-4481:
------------------------------------
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.
> CPP API should not initialize all handles on every event.
> ---------------------------------------------------------
>
> Key: TS-4481
> URL: https://issues.apache.org/jira/browse/TS-4481
> Project: Traffic Server
> Issue Type: Bug
> Components: CPP API
> Reporter: Alan M. Carroll
> Assignee: Brian Geffon
>
> To avoid problems with stale handles to transaction headers the CPP API was
> changed to initialize all such handles for a set of events. However this
> causes the error log to be flooded because not all headers are available for
> all of the events and each one that is not generates an error message. This
> generates a lot of errors even in normal use without problems. Simply
> disabling the messages seems a poor choice as the cases where the particular
> header is accessed and not found should be noted.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
