Updated example in java to use partitions#2160
Updated example in java to use partitions#2160mahnushm wants to merge 25 commits intoajewell/bucketsfrom
Conversation
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Minor error
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Adding GetNumberOfQueries and defaultNumberOfPartitions.
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Change i to partition
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Adding updatedQueryRequest
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
| .build(); | ||
|
|
||
|
|
||
| // The number can be obtained using transformClient.getNumberOfQueries(query) |
There was a problem hiding this comment.
You might want to be specific on what number can be obtained. Number of queries?
There was a problem hiding this comment.
Good point, I will add transformClient.getNumberOfQueries(query) to show its use case too.
|
@mahnushm and @rishav-karanjit, I noticed you are updating the smithy model files. |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Testing a value that is not power of 2
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Updated maximumNumberOfPartitions to 4 and clarified defaultNumberOfPartitions constraints.
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
| } | ||
| else { | ||
| // Adding the result for this partition | ||
| allResults.addAll(queryResponse.items()); |
There was a problem hiding this comment.
What would happen if response is paginated and has more results?
|
@mahnushm and @sharmabikram, I noticed you are updating the smithy model files. |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Updated number of partitions for beacons and adjusted related logic to ensure correct handling of query results.
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
Removed unused imports related to QueryInputTransform.
|
Detected changes to the release files or to the check-files action |
|
Changes to the release files or the check-files action requires 2 approvals from CODEOWNERS |
3 participants
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.