[SERVER-17573] move collection creation OpObserver::onCreateCollection down into createCollection, et cetera Created: 12/Mar/15 Updated: 02/Dec/15 Resolved: 09/Apr/15 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Code |
| Affects Version/s: | None |
| Fix Version/s: | 3.1.2 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Eric Milkie | Assignee: | Matt Dannenberg |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||
| Sprint: | RPL 1 04/03/15, RPL 2 04/24/15 | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Description |
|
To ensure no implicit collection creation forgets to explicitly log it in the oplog. |
| Comments |
| Comment by Githook User [ 09/Apr/15 ] |
|
Author: {u'username': u'dannenberg', u'name': u'matt dannenberg', u'email': u'matt.dannenberg@10gen.com'}Message: |
| Comment by Eric Milkie [ 13/Mar/15 ] |
|
Yes, it can be. The others are: Also, mr.cpp:460 I noticed is calling onInsert() but should be onCreateIndex(). |
| Comment by Andy Schwerin [ 12/Mar/15 ] |
|
Can this be an umbrella ticket for moving all "operation observation" into the catalog objects, instead of having to know every time you do a catalog operation whether or not you have to log it? I suppose index creation might not fit well into this design, right now. Perhaps another thing to think about as we think about how we instruct the system to build indexes. |