[SERVER-22057] Refactor OpObserver::onUpdate, pass updated document directly as parameter Created: 04/Jan/16 Updated: 05/Feb/16 Resolved: 15/Jan/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 3.3.1 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Dianna Hohensee (Inactive) | Assignee: | Dianna Hohensee (Inactive) |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Sprint: | Sharding E (01/08/16), Sharding F (01/29/16) |
| Participants: |
| Description |
|
Pass updated doc directly into onUpdate when it's available to the caller, so it doesn't have to be refetched from memory/disk later in the sharding logOp code. |
| Comments |
| Comment by Githook User [ 15/Jan/16 ] | ||||||||
|
Author: {u'username': u'DiannaHohensee', u'name': u'Dianna Hohensee', u'email': u'dianna.hohensee@10gen.com'}Message: | ||||||||
| Comment by Scott Hernandez (Inactive) [ 04/Jan/16 ] | ||||||||
|
Great, when you do please also include the why/use-case so there is some context for this change. | ||||||||
| Comment by Dianna Hohensee (Inactive) [ 04/Jan/16 ] | ||||||||
|
You're right | ||||||||
| Comment by Scott Hernandez (Inactive) [ 04/Jan/16 ] | ||||||||
|
Seems like it is already passed in...
|