Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Drop asyncJavaPaginatedRequest #168

Open
vigoo opened this issue Dec 11, 2020 · 1 comment
Open

Drop asyncJavaPaginatedRequest #168

vigoo opened this issue Dec 11, 2020 · 1 comment
Assignees
@vigoo

Comments

@vigoo
Copy link
Collaborator

vigoo commented Dec 11, 2020

Drop asyncJavaPaginatedRequest if possible and use asyncSimplePaginatedRequest instead. This would make all pagination unified and run the aspects for each pagination call not just the top level one.
@vigoo
Copy link
Collaborator Author

vigoo commented Dec 14, 2020

Note that it is not enough to simply just remove the support for the Java paginated requests, the native has to be improved to not only support the NextToken interface but custom ones defined in the paginator schema too, as for example query in https://github.com/aws/aws-sdk-java-v2/blob/master/services/dynamodb/src/main/resources/codegen-resources/dynamodb/paginators-1.json

@vigoo vigoo self-assigned this Mar 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vigoo vigoo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vigoo