Bug 27059 - Adding elapsed times/durations to updatedatabase would be nice.
Summary: Adding elapsed times/durations to updatedatabase would be nice.
Status: CLOSED WONTFIX
Alias: None
Product: Koha
Classification: Unclassified
Component: Installation and upgrade (command-line installer) (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Martin Renvoize (ashimema)
QA Contact: Testopia
URL:
Keywords:
Depends on: 24696
Blocks:
  Show dependency tree / graph
 
Reported: 2020-11-19 15:34 UTC by Martin Renvoize (ashimema)
Modified: 2022-12-12 21:25 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 27059: Add elapsed times to updatedatabase.pl (2.86 KB, patch)
2020-11-19 16:06 UTC, Martin Renvoize (ashimema) 		Details | Diff | Splinter Review
Bug 27059: Add elapsed times to updatedatabase.pl (2.98 KB, patch)
2020-11-20 15:34 UTC, Martin Renvoize (ashimema) 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Renvoize (ashimema) 2020-11-19 15:34:53 UTC 
In bug 24696 we added completion times to each step in updatedatabase, it would also be useful to display the elapsed times or time taken per step.
Comment 1 Martin Renvoize (ashimema) 2020-11-19 16:06:34 UTC 
Created attachment 113826 [details] [review]
Bug 27059: Add elapsed times to updatedatabase.pl
Comment 2 Martin Renvoize (ashimema) 2020-11-20 15:34:46 UTC 
Created attachment 113896 [details] [review]
Bug 27059: Add elapsed times to updatedatabase.pl

This patch adds an elapsed time to the timestamp field for each update.

Test plan
1/ Apply patch
2/ Run updatedatebase
3/ Note the addition of an elapsed time in the output
Comment 3 Fridolin Somers 2021-06-16 15:23:39 UTC 
Mmm does not work :

$ ./src/installer/data/mysql/updatedatabase.pl 
Use of uninitialized value $start_time in integer subtraction (-) at ./src/installer/data/mysql/updatedatabase.pl line 24474.
Upgrade to 20.12.00.045 done [17:22:25 (451071h:22m:25s)]: Bug 21249 - Adding new system preference SearchLimitLibrary
Upgrade to 20.12.00.046 done [17:22:25 (00h:00m:00s)]: Bug 14723 - Additional delivery notes to messages
Upgrade to 20.12.00.047 done [17:22:25 (00h:00m:00s)]: Bug 23215 - Remove core PayPal support in favor of the use of plugins
Upgrade to 20.12.00.048 done [17:22:25 (00h:00m:00s)]: Bug 26995 - [SKIP] Drop column relationship from borrower tables [not executed]
Upgrade to 20.12.00.049 done [17:22:25 (00h:00m:00s)]: Bug 28108 - Move action logs 'SERIAL CLAIM' and 'ACQUISITION CLAIM' to a new 'CLAIMS' module
Upgrade to 20.12.00.050 done [17:22:25 (00h:00m:00s)]: Bug 28108 - Add new systempreference OpacHiddenItemsHidesRecord