Skip to content
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

Log time taken to process a bill run #163

Merged
merged 3 commits into from
Mar 14, 2023
Merged

Conversation

Cruikshanks
Copy link
Member

@Cruikshanks Cruikshanks commented Mar 14, 2023

https://eaflood.atlassian.net/browse/WATER-3906

When issues of performance came up in the charging-module-api we found it immensely useful to have evidence to hand of how long the most contentious process was actually taking.

As we are not in control of the full process with our new SROC supplementary billing process, it would be useful to know exactly how long are part of the journey is taking. We're also about to expand the process to include reversing the previous supplementary bill run. So, it would be useful to get a working idea of how that impacts our timings.

To be clear, this is a very basic metric; the difference between the times at the start and the end of the process. There is no call at this time for factoring the size of a bill run, load on the servers at the time etc. It's all we needed in the Charging Module API, which also means it's some code we can readily steal!

Screenshot of log output

Screenshot 2023-03-14 at 09 34 06

https://eaflood.atlassian.net/browse/WATER-3906

When issues of performance came up in the [charging-module-api](https://github.com/DEFRA/sroc-charging-module-api) we found it immensely useful to have evidence to hand of how long the most contentious process was actually taking.

As we are not in control of the full process with our new SROC supplementary billing process, it would be useful to know exactly how long are part of the journey is taking. We're also about to expand the process to include reversing the previous supplementary bill run. So, it would be useful to get a working idea of how that impacts our timings.

To be clear, this is a very basic metric; the difference between the times at the start and the end of the process. There is no call at this time for factoring the size of a bill run, load on the servers at the time etc. It's all we needed in the Charging Module API, which also means it's some code we can readily steal!
@Cruikshanks Cruikshanks added the enhancement New feature or request label Mar 14, 2023
@Cruikshanks Cruikshanks self-assigned this Mar 14, 2023
We also include a test to document and demonstrate we are logging the time taken.
@Cruikshanks Cruikshanks marked this pull request as ready for review March 14, 2023 10:22
Copy link
Contributor

@Jozzey Jozzey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼

@Cruikshanks Cruikshanks merged commit 90136f4 into main Mar 14, 2023
@Cruikshanks Cruikshanks deleted the add-process-bill-run-timings branch March 14, 2023 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants