Skip to content

DATAMONGO-1208 - mongoTemplate.stream does not set limit #297

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

domenique
Copy link
Contributor

Reused the QueryCursorPreparer to apply skip, limit, sort, etc. to the cursor.

Reused the QueryCursorPreparer to apply skip, limit, sort, etc. to the cursor.
@thomasdarimont
Copy link

Hi Domenique,

thanks for your PR, I'll polish it a bit and apply it asap.
Would you mind signing the CLA and paste your CLA number here?

Cheers,
Thomas

thomasdarimont pushed a commit that referenced this pull request May 21, 2015
We now use the QueryCursorPreparer honor skip, limit, sort, etc. for streaming.

Original pull request: #297.
thomasdarimont pushed a commit that referenced this pull request May 21, 2015
We now use the QueryCursorPreparer honor skip, limit, sort, etc. for streaming.

Original pull request: #297.
Polished pull request: #298.
@domenique
Copy link
Contributor Author

Hi Thomas,

I've signed it yesterday but didn't really know what to do with it. Anyway, here it is: 121920150520061605

Krgds
Domenique

odrotbohm pushed a commit that referenced this pull request May 21, 2015
We now use the QueryCursorPreparer honor skip, limit, sort, etc. for streaming.

Original pull request: #297.
Polishing pull request: #298.
odrotbohm pushed a commit that referenced this pull request May 21, 2015
We now use the QueryCursorPreparer honor skip, limit, sort, etc. for streaming.

Original pull request: #297.
Polishing pull request: #298.
@odrotbohm
Copy link
Member

That's all we need, Domenique! Thanks a lot for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants