CiscoTestAutomation / ncdiff

NETCONF Diff Engine
Apache License 2.0
24 stars 5 forks source link

Remove memory leak introduced in #2 #13

Closed wouterdb closed 3 years ago

wouterdb commented 3 years ago

Hi all,

In #2 I introduced caches to improve performance. We have discovered that these caches introduce memory leaks.

Hereby a patch to fix this leak.

Details

We also filed a bug for this on python: https://bugs.python.org/issue44310

Backstory

edvgui commented 2 years ago

Hello @yuekyang ! Would it be possible to release this change? This is a pretty big deal and would help us a lot.

Thanks