[SERVER-33182] Remove atomic applyOps Created: 08/Feb/18 Updated: 29/Oct/23 Resolved: 10/Nov/22 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication |
| Affects Version/s: | None |
| Fix Version/s: | 6.2.0-rc0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Siyuan Zhou | Assignee: | Adi Agrawal |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | former-quick-wins, tech-debt-repl | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Sprint: | QE 2022-10-31, QE 2022-11-14 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Linked BF Score: | 0 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
All the 3.6 uses of applyOps are only on the same mongod. The use on config servers has been replaced with doTxn and will eventually be replaced with multi-statement transactions, so we should be able to remove atomic applyOps in 4.0. |
| Comments |
| Comment by Githook User [ 10/Nov/22 ] |
|
Author: {'name': 'Adityavardhan Agrawal', 'email': 'aa729@cornell.edu', 'username': 'Adityav369'}Message: |
| Comment by Louis Williams [ 27/Jul/22 ] |
|
This ticket is unblocked now. |
| Comment by Spencer Brody (Inactive) [ 04/Apr/18 ] |
|
I'd be happy to remove the (already quite limited) public documentation for applyOps. FYI steve.briskin behackett |
| Comment by Alyson Cabral (Inactive) [ 04/Apr/18 ] |
|
Spoke to Siyuan in person when this ticket was first opened, but to reiterate that convo, I am unaware of any external usage. We should probably remove the documentation of internal commands but I'll open a docs ticket on that. Any objections? |
| Comment by Spencer Brody (Inactive) [ 29/Mar/18 ] |
|
I think that everything needed to do That said, if we don't get it done in time for 4.0, we can do |
| Comment by Gregory McKeon (Inactive) [ 29/Mar/18 ] |
|
spencer, |
| Comment by Shane Harvey [ 09/Feb/18 ] |
|
mongomirror does not rely on atomic applyOps. It actually goes through hoops to avoid it. |
| Comment by Andy Schwerin [ 08/Feb/18 ] |
|
alyson.cabral, do you know if anyone uses the atomic applyOps behavior? The documentation says it's "internal" and makes no mention of the atomic form. |