diff options
author | David Pursehouse <dpursehouse@collab.net> | 2019-07-08 16:04:57 +0900 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2019-07-08 07:04:57 +0000 |
commit | 26b2ca99a096e81a9e495162d62022b2512910a4 (patch) | |
tree | 121222b201bf80ee70abe52e964983b6c68ab63b | |
parent | 8827498933138346e9c4fc223c6ccee3ae0d505f (diff) |
Update git submodules
* Update plugins/replication from branch 'stable-2.16'
to d8cc3f9e293d4fea1131e23d7ad9fac51d63589b
- Merge branch 'stable-2.15' into stable-2.16
* stable-2.15:
Add missing log statement if replication was canceled
When rescheduling due to in-flight push log also the in-flight task ID
Make sure to always remove in-flight pushes
Change-Id: I938d5ad49e94865515d8e2fe4694da8a605fb53a
- Add missing log statement if replication was canceled
Change-Id: Id06bae985cfdb000c66743ccd3c5d9f56d34d681
- When rescheduling due to in-flight push log also the in-flight task ID
When a push operation was rescheduled due to an in-flight push we logged
a line like:
[79bda07a] Rescheduling replication to https://host/foo.git to avoid collision with an in-flight push.
This log was already useful but it didn't tell us which push task was
actually in-flight.
Add the ID of the in-flight push task to the log:
[79bda07a] Rescheduling replication to https://host/foo.git to avoid collision with the in-flight push [80ceb18b].
This information is especially useful when a task gets repeatedly
rescheduled and we need to identify the root cause of that.
(cherry picked from commit f585eba0d21be222404b871bfe7a2b11e578eb14)
Change-Id: I7866f964cab1e4f479b0d7d62ae4aac3019fab4b
- Make sure to always remove in-flight pushes
When a push is finished successfully or fails due to an error we have to
remove it from the map of in-flight pushes. Although this was done from
a finally block, the same block performed a Repository.close() operation
prior to the in-flight push removal. If the Repository.close() threw an
exception the pool.notifyFinished() wouldn't be called and we would end
up with a zombie in-flight push.
Bug: Issue 10852
Change-Id: I8d5918d5271ba74ce12153f054503adaef155c5c
(cherry picked from commit 6fcdcf98fe61e8c6258fb40967f0cff6b8c38fd4)
m--------- | plugins/replication | 0 |
1 files changed, 0 insertions, 0 deletions
diff --git a/plugins/replication b/plugins/replication -Subproject 6fcdcf98fe61e8c6258fb40967f0cff6b8c38fd +Subproject d8cc3f9e293d4fea1131e23d7ad9fac51d63589 |