diff options
Diffstat (limited to 'storage')
| -rw-r--r-- | storage/src/vespa/storage/distributor/operations/idealstate/mergeoperation.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/storage/src/vespa/storage/distributor/operations/idealstate/mergeoperation.cpp b/storage/src/vespa/storage/distributor/operations/idealstate/mergeoperation.cpp index 7008cb36666..69bbb67b1f3 100644 --- a/storage/src/vespa/storage/distributor/operations/idealstate/mergeoperation.cpp +++ b/storage/src/vespa/storage/distributor/operations/idealstate/mergeoperation.cpp @@ -248,9 +248,9 @@ MergeOperation::deleteSourceOnlyNodes( _ok = _removeOperation->ok(); done(); } + } else { + done(); } - // FIXME what about the else-case here...? done() is not invoked by caller in this branch. - // Not calling done() doesn't leak anything, but causes metric updates to be missed. } void |