-
Notifications
You must be signed in to change notification settings - Fork 156
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
Support pagination for Deployments #532
Conversation
Co-authored-by: Ryo Narita <mail.ryonarita@gmail.com>
Code coverage for golang is
|
Fixed |
/lgtm |
Code coverage for golang is
|
Resolved the conflict. |
Yes, I was concerned about that. But It is not worth the effort for now. (After adding the support for MongoDB and DynamoDB we can reinvestigate about the pagination for all of them or can see how to run multiple queries with the same snapshot.) |
/approve |
Got it.
Absolutely. |
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #368
Does this PR introduce a user-facing change?: