xuemingqiang/mongo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

comment minor

Browse Source
This commit is contained in:
Dwight
2009-10-14 11:01:29 -04:00
parent 25d2f88e1e
commit b5a86240a9
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
db/repl.cpp
View File

@@ -1210,14 +1210,10 @@ namespace mongo {
{
time_t saveLast = time(0);
while ( 1 ) {
/* TODO: WHY is c->more() called twice here? Once in this block it's clearly false?
Also: isn't it bad to call more() without a temprelease? seems like we need
another helper haveMoreWithoutRequesting(), and then if that is false, we temp
release and try more().
*/
/* todo this code is ugly and confusing fix */
if ( !c->more() ) {
dblock lk;
OpTime nextLastSaved = nextLastSavedLocalTs();
OpTime nextLastSaved = nextLastSavedLocalTs(); // this may make c->more() become true
{
dbtemprelease t;
if ( c->more() ) {