-
Notifications
You must be signed in to change notification settings - Fork 274
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
Preventing retentionTime and retentionCount to be set to 0 #2790
Preventing retentionTime and retentionCount to be set to 0 #2790
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2790 +/- ##
============================================
+ Coverage 64.24% 70.22% +5.98%
- Complexity 10398 11678 +1280
============================================
Files 840 840
Lines 71755 71920 +165
Branches 8611 8647 +36
============================================
+ Hits 46099 50509 +4410
+ Misses 23004 18775 -4229
+ Partials 2652 2636 -16 ☔ View full report in Codecov by Sentry. |
df3f8e0
to
ca2c50a
Compare
ca2c50a
to
42ed2a2
Compare
if (ttlFromHeader == 0) { | ||
throw new RestServiceException(Headers.TTL + "[" + ttlFromHeader + "] should not be 0", | ||
RestServiceErrorCode.InvalidArgs); | ||
} |
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.
we probably don't need this one here.
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.
Remove this part and updated to add test and filter out null(default) after serialization as we discussed.
9df34a8
to
687b83c
Compare
687b83c
to
31edd19
Compare
field.setAccessible(true); | ||
Object value = field.get(dataset); | ||
if (value != null) { | ||
gen.writeObjectField(field.getName(), value); |
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.
we should probably use annotation value here. something like
gen.writeObjectField(field.getAnnotation(JsonProperty.class).value(), value);
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.
Sure, updated it.
No description provided.