-
Notifications
You must be signed in to change notification settings - Fork 605
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
bench/db: update ft_transfers
schema
#11563
Conversation
@@ -121,6 +121,7 @@ def commit_to_db(data: dict) -> None: | |||
commit_hash, commit_time = get_commit() | |||
response = { | |||
"time": time_begin.strftime('%Y-%m-%dT%H:%M:%SZ'), | |||
"time_end": datetime.now().strftime('%Y-%m-%dT%H:%M:%SZ'), |
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.
@MCJOHN974 can you verify if this is the correct way to add time_end
to the response
?
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.
Yes I think it will work
benchmarks/continous/db/tool/orm/migrations/2024-06-12-120242_refine_ft_transfers/down.sql
Outdated
Show resolved
Hide resolved
benchmarks/continous/db/tool/orm/migrations/2024-06-12-120242_refine_ft_transfers/down.sql
Show resolved
Hide resolved
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #11563 +/- ##
==========================================
- Coverage 71.48% 71.47% -0.01%
==========================================
Files 788 788
Lines 160835 160835
Branches 160835 160835
==========================================
- Hits 114968 114956 -12
- Misses 40850 40863 +13
+ Partials 5017 5016 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Sorry, missed this!
@akashin, @MCJOHN974 could you please take another look, I've added a new migration as discussed on Zulip:
|
- Adds a `time_end` field to allow recording both the start and end time of benchmark runs. This was requested in an issue which I couldn't find anymore in a quick search. - Values of `total_transactions` and `size_state_bytes` might get out of the `integer` range, hence the type of these columns is changed to `bigint`.
time_end
field to allow recording both the start and end time of benchmark runs. This was requested in an issue which I couldn't find anymore in a quick search.total_transactions
andsize_state_bytes
might get out of theinteger
range, hence the type of these columns is changed tobigint
.