Skip to content
This repository was archived by the owner on Feb 12, 2022. It is now read-only.

Conversation

msgroi
Copy link
Collaborator

@msgroi msgroi commented Sep 10, 2018

This is for review (with @jeremy-horwitz ) purposes only, no plan to merge just yet.

…les and underlying CreateTableRequestFactory useable outside of the SharedTableBuilder
@coveralls
Copy link

coveralls commented Sep 10, 2018

Pull Request Test Coverage Report for Build 335

  • 23 of 65 (35.38%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.0%) to 83.938%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/com/salesforce/dynamodbv2/mt/mappers/sharedtable/HybridSharedTableBuilder.java 0 42 0.0%
Totals Coverage Status
Change from base Build 319: -2.0%
Covered Lines: 1505
Relevant Lines: 1793

💛 - Coveralls

@msgroi msgroi requested a review from horwitzj September 11, 2018 16:00
@msgroi
Copy link
Collaborator Author

msgroi commented Sep 12, 2018

Reviewed with @horwitzj offline.

@msgroi msgroi merged commit b808a12 into master Sep 12, 2018
@msgroi msgroi deleted the zos-mt-dynamo-v2 branch September 12, 2018 00:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants