-
Notifications
You must be signed in to change notification settings - Fork 910
DDB Enhanced: updates README with bean examples #1731
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.
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
Conversation
converterProvider1.class, | ||
converterProvider2.class, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Capital C?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:-)
You can flatten as many different eligible classes as you like using the flatten annotation. | ||
The only constraints are that attributes must not have the same name when they are being rolled | ||
together, and there must never be more than one partition key, sort key or table name. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should move this text underneath the StaticTableSchema example.
Codecov Report
@@ Coverage Diff @@
## master #1731 +/- ##
============================================
+ Coverage 76.24% 76.27% +0.03%
Complexity 182 182
============================================
Files 1070 1070
Lines 32225 32225
Branches 2520 2520
============================================
+ Hits 24569 24579 +10
+ Misses 6415 6404 -11
- Partials 1241 1242 +1
Continue to review full report at Codecov.
|
e67302b
to
5eb3c74
Compare
Kudos, SonarCloud Quality Gate passed!
|
…a56ad676b Pull request: release <- staging/4e91a7a2-23ac-4046-93b8-dada56ad676b
Description
Adds sections for the StaticTableSchema and BeanTableSchema, including usage of