search

derjust / spring-data-dynamodb

This module deals with enhanced support for a data access layer built on AWS DynamoDB.
https://derjust.github.io/spring-data-dynamodb/
Apache License 2.0
403 stars 141 forks source link

Added Query and Scan limit features to save money on the new ondemand… #239

Closed derjust closed 5 years ago

derjust commented 5 years ago

… feature

codecov-io commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9c2d8a7). Click here to learn what that means. The diff coverage is 62.96%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #239   +/-   ##
=========================================
  Coverage          ?   67.28%           
  Complexity        ?      658           
=========================================
  Files             ?       84           
  Lines             ?     2253           
  Branches          ?      394           
=========================================
  Hits              ?     1516           
  Misses            ?      556           
  Partials          ?      181
Impacted Files Coverage Δ Complexity Δ
.../repository/support/DynamoDBEntityInformation.java 50% <ø> (ø) 1 <0> (?)
...in/spring/data/dynamodb/core/DynamoDBTemplate.java 68.05% <0%> (ø) 16 <0> (?)
...pring/data/dynamodb/repository/QueryConstants.java 0% <0%> (ø) 0 <0> (?)
...y/query/DynamoDBEntityWithHashKeyOnlyCriteria.java 69.04% <100%> (ø) 13 <0> (?)
...namodb/repository/query/PartTreeDynamoDBQuery.java 85.71% <100%> (ø) 10 <0> (?)
...epository/query/AbstractDynamoDBQueryCriteria.java 68.46% <100%> (ø) 101 <1> (?)
...ynamodb/repository/query/DynamoDBQueryCreator.java 75% <100%> (ø) 2 <1> (?)
...ery/DynamoDBEntityWithHashAndRangeKeyCriteria.java 74.3% <100%> (ø) 74 <0> (?)
...db/repository/query/DynamoDBCountQueryCreator.java 44.44% <50%> (ø) 2 <1> (?)
...repository/query/AbstractDynamoDBQueryCreator.java 49.05% <50%> (ø) 11 <0> (?)
... and 3 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 9c2d8a7...709d9a2. Read the comment docs.

codecov-io commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9c2d8a7). Click here to learn what that means. The diff coverage is 62.96%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #239   +/-   ##
=========================================
  Coverage          ?   67.28%           
  Complexity        ?      658           
=========================================
  Files             ?       84           
  Lines             ?     2253           
  Branches          ?      394           
=========================================
  Hits              ?     1516           
  Misses            ?      556           
  Partials          ?      181
Impacted Files Coverage Δ Complexity Δ
.../repository/support/DynamoDBEntityInformation.java 50% <ø> (ø) 1 <0> (?)
...in/spring/data/dynamodb/core/DynamoDBTemplate.java 68.05% <0%> (ø) 16 <0> (?)
...pring/data/dynamodb/repository/QueryConstants.java 0% <0%> (ø) 0 <0> (?)
...y/query/DynamoDBEntityWithHashKeyOnlyCriteria.java 69.04% <100%> (ø) 13 <0> (?)
...namodb/repository/query/PartTreeDynamoDBQuery.java 85.71% <100%> (ø) 10 <0> (?)
...epository/query/AbstractDynamoDBQueryCriteria.java 68.46% <100%> (ø) 101 <1> (?)
...ynamodb/repository/query/DynamoDBQueryCreator.java 75% <100%> (ø) 2 <1> (?)
...ery/DynamoDBEntityWithHashAndRangeKeyCriteria.java 74.3% <100%> (ø) 74 <0> (?)
...db/repository/query/DynamoDBCountQueryCreator.java 44.44% <50%> (ø) 2 <1> (?)
...repository/query/AbstractDynamoDBQueryCreator.java 49.05% <50%> (ø) 11 <0> (?)
... and 3 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 9c2d8a7...709d9a2. Read the comment docs.

derjust commented 5 years ago

Adding test case for #232