Abhijit so in the master branch no problem we got plenty of time, but you say for the SR3 better with race condition than regression ?
P.S.: Right now all merges are blocked assume so we can only prepare revert. Here is the revert for boron: https://git.opendaylight.org/gerrit/#/c/53953/ Jozef. From: Abhijit Kumbhare [mailto:[email protected]] Sent: Tuesday, March 28, 2017 7:21 AM To: Luis Gomez <[email protected]>; openflowplugin-dev <[email protected]> Subject: Re: [openflowplugin-dev] Last patch in boron introduces perf regression Sounds good. Jozef/Tomas can you do it? I assume the race condition that the patch fixed was pre-existing. On Mon, Mar 27, 2017 at 9:12 PM Luis Gomez <[email protected]<mailto:[email protected]>> wrote: Hi all, After looking these boron jobs: https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-1node-periodic-scale-stats-collection-daily-only-boron/ https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-3node-periodic-bulkomatic-clustering-daily-only-boron/ and testing SR3 manually, I conclude last patch we merged to fix stats issues: https://git.opendaylight.org/gerrit/#/c/53545 Introduces *important* perf regression, so I really recommend to revert it before we release SR3. Same thing I recommend for master but there we have more time to revert/fix: https://git.opendaylight.org/gerrit/#/c/52237 BR/Luis _______________________________________________ openflowplugin-dev mailing list [email protected]<mailto:[email protected]> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
