[JAVA-4467] DefaultServerMonitor.connect must signal all threads (monitor and RTT) rather than signalling one of them at random Created: 01/Feb/22 Updated: 27/Oct/23 Resolved: 03/Feb/22 |
|
| Status: | Closed |
| Project: | Java Driver |
| Component/s: | Monitoring |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Valentin Kavalenka | Assignee: | Valentin Kavalenka |
| Resolution: | Works as Designed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Documentation Changes: | Not Needed |
| Description |
|
DefaultServerMonitor.condition is used in waitForNext, which is then used by both the monitor thread and the RTT thread. Currently DefaultServerMonitor.connect calls condition.signal, which means that only one of the two threads is resumed, instead of resuming both. |
| Comments |
| Comment by Valentin Kavalenka [ 03/Feb/22 ] |
|
See this comment by jeff.yemin on why it is not a bug. |