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

[couchbase] add debug-level phase timings. #4466

Merged
merged 1 commit into from
Sep 20, 2021

Conversation

daschl
Copy link
Member

@daschl daschl commented Sep 17, 2021

This changeset adds individual phase timings at debug level so
we have actual numbers to figure out where to start optimizing
the whole bootstrap process.

This changeset adds individual phase timings at debug level so
we have actual numbers to figure out where to start optimizing
the whole bootstrap process.
@kiview
Copy link
Member

kiview commented Sep 20, 2021

Hey @daschl,
cool, LGTM. Can you give a bit more context about how is mainly profiting from this PR? Is this something that can help Couchbase to improve Docker images with regards to Testcontainer usage, or is this more for maintainers of this module?

@daschl
Copy link
Member Author

daschl commented Sep 20, 2021

It is for the maintainers of the module - I want to make it faster over time and this is the first step in actually gathering numbers.

@kiview
Copy link
Member

kiview commented Sep 20, 2021

Alright, expected something like this 👍

@rnorth rnorth merged commit 28d74da into testcontainers:master Sep 20, 2021
@kiview kiview added this to the next milestone Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants