[SERVER-27758] replBatchLimitOperations should be unsigned Created: 19/Jan/17  Updated: 06/Dec/22  Resolved: 17/Apr/20

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

Type: Bug Priority: Major - P3
Reporter: Eric Milkie Assignee: Backlog - Replication Team
Resolution: Won't Fix Votes: 0
Labels: neweng
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Assigned Teams:
Replication
Operating System: ALL
Participants:

 Description   

In sync_tail.h:

static AtomicInt32 replBatchLimitOperations;

This should instead be an AtomicWord<size_t>, because we use it to initialize variables of size_t type (which is unsigned), and because it should match the type of the variable with the same name found in data_replicator.h. (We should probably change replBatchLimitOperations in data_replicator.h to be std::size_t instead of std::uint32_t as well)



 Comments   
Comment by Judah Schvimer [ 17/Apr/20 ]

This has a validator of >=1 and <= 1,000,000, so I don't think this is a priority anymore.

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