[
https://issues.apache.org/jira/browse/TS-4481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15300137#comment-15300137
]
ASF GitHub Bot commented on TS-4481:
------------------------------------
Github user SolidWallOfCode commented on the pull request:
https://github.com/apache/trafficserver/pull/663#issuecomment-221594161
Unfortunately `event_` is actually used in `Transaction::initCachedRequest`
to do handle initialization differently for `TS_EVENT_HTTP_TXN_CLOSE`. I don't
understand the implications well enough to say if that's important.
As for removing the handle caching stuff, at this point that's effectively
what I've done. We could just move the content of the init methods in to the
get methods since the latter are just wrappers for the former.
> 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)
