[PROTON-1709] [python] ApplicationEvent causing memory growth

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: proton-c-0.18.1
• Fix Version/s: proton-c-0.31.0
• Component/s: python-binding
• Labels: leak
• Assignee: Andrew Stitcher
• Reporter: Xin Chen
• Created: 01/Dec/17 22:55
• Updated: 14/May/20 09:30
• Resolved: 12/Dec/19 01:44

Description

ApplicationEvent creates a new EventType object in the constructor. The EventType object adds itself to the TYPES dict always, even for the same typename. There is no way for the application to avoid this by providing an EventType object directly. Repro code below:

import threading
from proton.reactor import Container, ApplicationEvent, EventInjector

class Program:
    def __init__(self, injector):
        self.injector = injector

    def on_reactor_init(self, event):
        event.reactor.selectable(self.injector)

    def on_hello(self, event):
        print(event.subject)

    def on_done(self, event):
        event.subject.stop()

e = EventInjector()
r = Container(Program(e))
t = threading.Thread(target=r.run)
t.start()

for i in range(1, 10000000):
    e.trigger(ApplicationEvent("hello", subject=str(i)))

e.trigger(ApplicationEvent("done", subject=r))

t.join()

Issue Links

• relates to: PROTON-2220 Add tests for recently fixed leaks in Python binding