[GitHub] nifi issue #2995: NIFI-5577 Fixing update method in NaiveRevisionManager so ...

2018-09-14 Thread markap14
Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/2995 @bbende thanks for the fix! All looks good to me. +1 merged to master. ---

[GitHub] nifi issue #2995: NIFI-5577 Fixing update method in NaiveRevisionManager so ...

2018-09-14 Thread markap14
Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/2995 Reviewing... ---

[GitHub] nifi issue #2995: NIFI-5577 Fixing update method in NaiveRevisionManager so ...

2018-09-06 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/2995 I believe the existing code in the finally block was there as a result of an older approach where locking was being performed in the revision manager, but since that is no longer needed, I believe it