[SERVER-42468] TransactionCoordinatorMetricsTest has race between transaction coordinator's lifetime and checking for log lines Created: 29/Jul/19 Updated: 29/Oct/23 Resolved: 29/Jul/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 4.2.0-rc5, 4.3.1 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Kaloian Manassiev | Assignee: | Kaloian Manassiev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Operating System: | ALL | ||||||||
| Backport Requested: |
v4.2
|
||||||||
| Sprint: | Sharding 2019-08-12 | ||||||||
| Participants: | |||||||||
| Linked BF Score: | 22 | ||||||||
| Description |
|
There is a race condition in the TransactionCoordinatorMetricsTests between when we stop capturing the test's log messages stream and the completion of the asynchronous completion of the transaction coordinator, which might not have gotten to log the slow transaction message by that time. In order to fix it, we should wait for the onCompletion notification on the TransactionCoordinator to fire before we stop recording the log messages. |
| Comments |
| Comment by Githook User [ 29/Jul/19 ] |
|
Author: {'name': 'Kaloian Manassiev', 'email': 'kaloian.manassiev@mongodb.com', 'username': 'kaloianm'}Message: (cherry picked from commit 25242cc39477596e23ae0ab4d4fd21b63a2f3ed6) |
| Comment by Githook User [ 29/Jul/19 ] |
|
Author: {'name': 'Kaloian Manassiev', 'email': 'kaloian.manassiev@mongodb.com', 'username': 'kaloianm'}Message: |