You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/01/05 21:37:58 UTC
[jira] [Work logged] (TS-4664) Crash due to separate event handlers
for IO events and plugin events for ClientSession
[ https://issues.apache.org/jira/browse/TS-4664?focusedWorklogId=34835&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-34835 ]
ASF GitHub Bot logged work on TS-4664:
--------------------------------------
Author: ASF GitHub Bot
Created on: 05/Jan/17 21:37
Start Date: 05/Jan/17 21:37
Worklog Time Spent: 10m
Work Description: Github user shinrich commented on the issue:
https://github.com/apache/trafficserver/pull/801
I'm actually holding off on this one. We are running our version of 5.3.x with this change, but there were a number of other changes as well. Not clear this one is needed. I plan to do stress testing of 7.x this month. I'll pull this logic forward if it is needed (or at least understand the issue better).
Issue Time Tracking
-------------------
Worklog Id: (was: 34835)
Time Spent: 2.5h (was: 2h 20m)
> Crash due to separate event handlers for IO events and plugin events for ClientSession
> --------------------------------------------------------------------------------------
>
> Key: TS-4664
> URL: https://issues.apache.org/jira/browse/TS-4664
> Project: Traffic Server
> Issue Type: Bug
> Reporter: Susan Hinrichs
> Assignee: Susan Hinrichs
> Fix For: 7.1.0
>
> Time Spent: 2.5h
> Remaining Estimate: 0h
>
> Found while tracking TS-4507 and original fix on that branch.
> Cleaned up handling regular events at the same time as plugin events. The original code relied on the subclasses overriding handle_api_event to handle the regular events, but the handler only handled the TIMEOUT event. Changed that to augment the subclasses' main event handler to call out to state_api_callout in the event of the plugin events.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)