Respect the pagination size for queries

[derjust]

This change respects the pagination size as provided from a Pageable
object given to a findXXX method.
Mostly important in conjunction with lazy list processing via

DynamoDBMapperConfig.Builder builder = new DynamoDBMapperConfig.Builder();
builder.setPaginationLoadingStrategy(PaginationLoadingStrategy.ITERATION_ONLY);

as this loads the result set of a query page-by-page.
Also see the comments in
com.amazonaws.services.dynamodbv2.datamodeling.PaginatedQueryList<T> as
used by
org.socialsignin.spring.data.dynamodb.core.DynamoDBTemplate.query(Class<T>, QueryRequest)

[derjust]

Use Pageable#unpaged() instead

[derjust]

Use Pageable#unpaged() instead

[derjust]

And update for the Wiki/Documentation is required

[codecov-io]

Codecov Report

Merging #117 into master will decrease coverage by 15.34%.
The diff coverage is 66.66%.

@@              Coverage Diff              @@
##             master     #117       +/-   ##
=============================================
- Coverage     64.29%   48.95%   -15.35%     
+ Complexity      549      424      -125     
=============================================
  Files            71       70        -1     
  Lines          1882     1859       -23     
  Branches        354      352        -2     
=============================================
- Hits           1210      910      -300     
- Misses          543      828      +285     
+ Partials        129      121        -8

Impacted Files	Coverage Δ	Complexity Δ
...ynamodb/repository/query/DynamoDBQueryCreator.java	60% <66.66%> (-20%)	2 <0> (ø)
...epository/query/AbstractDynamoDBQueryCriteria.java	67.71% <66.66%> (-1.35%)	103 <2> (+5)
...ory/config/DynamoDBRepositoryNameSpaceHandler.java	0% <0%> (-100%)	0% <0%> (-2%)
...pository/config/DynamoDBRepositoriesRegistrar.java	0% <0%> (-100%)	0% <0%> (-3%)
.../dynamodb/mapping/event/AuditingEventListener.java	0% <0%> (-100%)	0% <0%> (-2%)
...ynamodb/mapping/DefaultDynamoDBDateMarshaller.java	0% <0%> (-100%)	0% <0%> (-1%)
...namodb/mapping/AbstractDynamoDBDateMarshaller.java	0% <0%> (-100%)	0% <0%> (-5%)
...ng/data/dynamodb/query/QueryRequestCountQuery.java	0% <0%> (-100%)	0% <0%> (-2%)
...data/dynamodb/query/QueryExpressionCountQuery.java	0% <0%> (-100%)	0% <0%> (-2%)
...ository/support/DynamoDBRepositoryFactoryBean.java	0% <0%> (-100%)	0% <0%> (-7%)
... and 51 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8523f4...437d47a. Read the comment docs.

[derjust]

Updated documentation at https://github.com/derjust/spring-data-dynamodb/wiki/Query-Size-Limits-and-Pageable

[derjust]

Second thought on org.socialsignin.spring.data.dynamodb.repository.query.AbstractDynamoDBQuery.SlicedExecution.readPageOfResultsRestrictMaxResultsIfNecessary(Iterator<T>, int) required

[gmiro]

@derjust - Do you plan on merging this PR anytime soon? Trying to decide if I should use the DDB mapper directly instead of relying on spring-data-dynamodb. Thank you!