https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/NETVIRT-886 netvirt <p>As found in see <a href="https://jira.opendaylight.org/browse/NETVIRT-883" title="Umbrella parent issue for grouping all suspected transaction leaks" class="issue-link" data-issue-key="NETVIRT-883"><del>NETVIRT-883</del></a> attachment, there are many thousands of (read only) transactions that are opened in StatisticsImpl but apparently never closed:<br/> org.opendaylight.netvirt.statistics.StatisticsImpl.handleInterfaceRemoval(StatisticsImpl.java:398)<br/> org.opendaylight.netvirt.statistics.CountersServiceInterfaceListener.remove(CountersServiceInterfaceListener.java:41)<br/> org.opendaylight.netvirt.statistics.CountersServiceInterfaceListener.remove(CountersServiceInterfaceListener.java:19)</p> <p>Operating System: All<br/> Platform: All</p> NETVIRT-886

Suspected ReadOnlyTransaction leak in StatisticsImpl

Bug Resolved Done Michael Vorburger Michael Vorburger Mon, 4 Sep 2017 17:03:21 +0000 Fri, 24 Nov 2017 13:18:07 +0000 Fri, 24 Nov 2017 13:18:07 +0000 Nitrogen Nitrogen Carbon Oxygen General 0 1 <p>This popped up in <a href="https://jira.opendaylight.org/browse/NETVIRT-985" title="java.lang.OutOfMemoryError: Java heap space" class="issue-link" data-issue-key="NETVIRT-985"><del>NETVIRT-985</del></a> again today, so I'm proposing <a href="https://git.opendaylight.org/gerrit/65562" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/65562</a> to fix it, as it seems pretty bad:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre> 2774x transactions opened here, which are not closed: 2531x transactions opened here, which are not closed: 2390x transactions opened here, which are not closed: org.opendaylight.controller.md.sal.binding.impl.BindingDOMDataBrokerAdapter.newReadOnlyTransaction(BindingDOMDataBrokerAdapter.java:59) (...) org.opendaylight.netvirt.statistics.StatisticsImpl.handleInterfaceRemoval(StatisticsImpl.java:398) org.opendaylight.netvirt.statistics.CountersServiceInterfaceListener.remove(CountersServiceInterfaceListener.java:41) </pre> </div></div> <p>c/65562 is for carbon - I'll attempt to cherry-pick to Nitrogen, and for master we could use <tt>ManagedNewTransactionRunner</tt>?</p> <p>&gt; for master we could use ManagedNewTransactionRunner?</p> <p>ended-up doing a POCP (Plain Old Cherry Pick) for this one to master as well, becase ManagedNewTransactionRunner didn't actually support ReadOnlyTransaction, yet (but it soon will, via <a href="https://git.opendaylight.org/gerrit/#/c/65565/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/65565/</a> and <a href="https://git.opendaylight.org/gerrit/#/c/65566/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/65566/</a>, but that shouldn't hold this up).</p> Blocks NETVIRT-883 NETVIRT-985 Development External issue ID 9103 External issue URL Rank 0|i01ukf: