[SERVER-5183] Get rid of setNoJournal Created: 02/Mar/12 Updated: 11/Jul/16 Resolved: 22/Mar/12 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication |
| Affects Version/s: | None |
| Fix Version/s: | 2.1.1 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Mathias Stearn | Assignee: | Andy Schwerin |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Participants: | |||||||||
| Description |
|
Actually pretty easy and I think it will fix part of the closeall.js test. Code review should be up soon. |
| Comments |
| Comment by auto [ 22/Mar/12 ] |
|
Author: {u'login': u'erh', u'name': u'Eliot Horowitz', u'email': u'eliot@10gen.com'}Message: fix local.slaves updating after |
| Comment by Eliot Horowitz (Inactive) [ 21/Mar/12 ] |
|
I think Any change will get it reset. |
| Comment by Eliot Horowitz (Inactive) [ 21/Mar/12 ] |
|
Yes. local.slaves is very important and we need the cache reset. |
| Comment by Andy Schwerin [ 21/Mar/12 ] |
|
Will work on it. Are you certain that anyone actually consults local.slaves? |
| Comment by Mathias Stearn [ 21/Mar/12 ] |
|
We need to either not clear cache on changes to local.slaves or filter out the changes due to cache flushing. I know how to do this, but I'll be on various planes for the next 24 hours or so (actually on a plane right now delayed at the gate.) Feel free to fix in the mean time. |
| Comment by Eric Milkie [ 21/Mar/12 ] |
|
This seems to have possibly broke getlasterror/write concern. |
| Comment by auto [ 14/Mar/12 ] |
|
Author: {u'login': u'erh', u'email': u'eliot@10gen.com', u'name': u'Eliot Horowitz'}Message: no one uses setNoJournal anymore so get rid of |
| Comment by auto [ 14/Mar/12 ] |
|
Author: {u'login': u'erh', u'name': u'Eliot Horowitz', u'email': u'eliot@10gen.com'}Message: Make SlaveTracking::update not update row directly |