-
Notifications
You must be signed in to change notification settings - Fork 504
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
Not allowed empty primary values #847
Conversation
Codecov Report
@@ Coverage Diff @@
## master #847 +/- ##
=============================================
- Coverage 71.16% 55.68% -15.49%
+ Complexity 4358 3368 -990
=============================================
Files 283 283
Lines 20967 20968 +1
Branches 2959 2960 +1
=============================================
- Hits 14921 11675 -3246
- Misses 4566 7975 +3409
+ Partials 1480 1318 -162
Continue to review full report at Codecov.
|
@@ -115,6 +115,8 @@ public String name() { | |||
"Primary values must not be empty " + | |||
"(has properties %s)", hasProperties()); | |||
this.name = SplicingIdGenerator.concatValues(propValues); | |||
E.checkArgument(!this.name.isEmpty(), | |||
"The primary values can't be empty"); |
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.
The value of primary key can't be empty
88454c5
to
bb22551
Compare
Change-Id: I6b7502892383d08348734502d0e59f5d94128fc7
bb22551
to
5047f2a
Compare
Change-Id: I6b7502892383d08348734502d0e59f5d94128fc7