SERVER-4150 set position of client cursor, to be returned by get more

mongodb · Feb 25, 2012 · a08868d · a08868d
1 parent 7d9d6ac
commit a08868d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/mongo/db/ops/query.cpp b/src/mongo/db/ops/query.cpp
@@ -1129,7 +1129,9 @@ namespace mongo {
             // at this point
             throw SendStaleConfigException( ns , "version changed during initial query" );
         }
-
+
+        long long nReturned = queryResponseBuilder.handoff( result );
+
         ccPointer.reset();
         long long cursorid = 0;
         if ( saveClientCursor ) {
@@ -1154,12 +1156,12 @@ namespace mongo {
                 exhaust = ns;
                 curop.debug().exhaust = true;
             }
+            ccPointer->setPos( nReturned );
             ccPointer->pq = pq_shared;
             ccPointer->fields = pq.getFieldPtr();
             ccPointer.release();
         }
 
-        long long nReturned = queryResponseBuilder.handoff( result );
         QueryResult *qr = (QueryResult *) result.header();
         qr->cursorId = cursorid;
         curop.debug().cursorid = ( cursorid == 0 ? -1 : qr->cursorId );
@@ -1401,7 +1403,7 @@ namespace mongo {
 
             cursorid = cc->cursorid();
             DEV tlog(2) << "query has more, cursorid: " << cursorid << endl;
-            cc->setPos( n );
+//            cc->setPos( n );
 //            cc->pq = pq_shared;
 //            cc->fields = pq.getFieldPtr();
 //            cc->originalMessage = m;

diff --git a/src/mongo/dbtests/querytests.cpp b/src/mongo/dbtests/querytests.cpp
@@ -208,6 +208,7 @@ namespace QueryTests {
                 ClientCursor::Pointer clientCursor( cursorId );
                 ASSERT( clientCursor.c()->pq );
                 ASSERT_EQUALS( 2, clientCursor.c()->pq->getNumToReturn() );
+                ASSERT_EQUALS( 2, clientCursor.c()->pos() );
             }
 
             cursor = client().getMore( ns, cursorId );