-
Notifications
You must be signed in to change notification settings - Fork 914
Add support for ByteBufferAttributeConverter #1911
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
836bee2
to
6783b42
Compare
Codecov Report
@@ Coverage Diff @@
## master #1911 +/- ##
============================================
+ Coverage 76.24% 76.27% +0.03%
Complexity 187 187
============================================
Files 1076 1077 +1
Lines 32573 32585 +12
Branches 2569 2569
============================================
+ Hits 24835 24854 +19
+ Misses 6478 6474 -4
+ Partials 1260 1257 -3
Continue to review full report at Codecov.
|
6783b42
to
e4d7ce3
Compare
Kudos, SonarCloud Quality Gate passed!
|
Hi, I am wondering if there is anything I can do to get this code change merged. I think it is a missing feature which is useful for everybody. Steven |
...ware/amazon/awssdk/enhanced/dynamodb/converters/attribute/BinaryAttributeConvertersTest.java
Show resolved
Hide resolved
e4d7ce3
to
a5ae4b9
Compare
Add support ByteBuffer as an an attribute type. The DynamoDBType already lists it as a standard type.
a5ae4b9
to
1922e0c
Compare
Kudos, SonarCloud Quality Gate passed! |
…282f4cf66 Pull request: release <- staging/e9450161-1693-4a2b-a029-4b7282f4cf66
Add support ByteBuffer as an an attribute type.
Description
The DynamoDBType already lists it as a standard type.
Motivation and Context
The DynamoDBType mentions it as a standard type, but it was not supported.
Testing
Added test
Types of changes
Checklist
mvn install
succeedsLicense