-
Notifications
You must be signed in to change notification settings - Fork 108
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
Change schedule primary key from consensus_timestamp to schedule_id #2382
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2382 +/- ##
============================================
- Coverage 84.35% 84.35% -0.01%
- Complexity 2308 2309 +1
============================================
Files 439 439
Lines 11982 11984 +2
Branches 1020 1020
============================================
+ Hits 10108 10109 +1
Misses 1556 1556
- Partials 318 319 +1
Continue to review full report at Codecov.
|
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
92872cf
to
b560191
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
on schedule (schedule_id desc, consensus_timestamp desc); | ||
|
||
alter table if exists schedule | ||
add primary key (schedule_id); |
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 may be missing a REST endpoint for filter by timestamp, which is where an additional timestamp index might be useful to speed up queries.
We can address that later though
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.
LGTM
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.
LGTM
Description:
Related issue(s):
Fixes #2380
Notes for reviewer:
Checklist