[SERVER-48142] Remove allowSkippingAppendRequiredFieldsToResponse fail-point Created: 12/May/20  Updated: 12/Dec/23

Status: Backlog
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Major - P3
Reporter: Amirsaman Memaripour Assignee: Backlog - Cluster Scalability
Resolution: Unresolved Votes: 0
Labels: neweng
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Gantt Dependency
has to be done after SERVER-48013 Use atomics to reduce contention in h... Closed
Assigned Teams:
Cluster Scalability
Backport Requested:
v4.4
Participants:

 Description   

Some of the sharding unit-tests (e.g., NoErrors) expect a different behavior fromĀ appendRequiredFieldsToResponse than what the normal execution requires. These tests never initialize LogicalTimeValidator, and expect appendRequiredFieldsToResponse to quick-return.

To fill this gap, SERVER-48013 introduced a fail-point to quick-return from these unit-tests. This fail-point enables successful execution of these unit-tests, but makes such tests observe a non-real behavior for appendRequiredFieldsToResponse.

This ticket should make appropriate changes to these unit-tests to obviate the fail-point, and then remove it.


Generated at Thu Feb 08 05:16:16 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.