Skip to content

tracing: log chosen tokio-console port in orchestrator-tracing#12785

Merged
guswynn merged 1 commit into
MaterializeInc:mainfrom
guswynn:log-tokio
Jun 1, 2022
Merged

tracing: log chosen tokio-console port in orchestrator-tracing#12785
guswynn merged 1 commit into
MaterializeInc:mainfrom
guswynn:log-tokio

Conversation

@guswynn

@guswynn guswynn commented May 31, 2022

Copy link
Copy Markdown
Contributor

Motivation

  • This PR refactors existing code.

Testing

  • This PR has adequate test coverage / QA involvement has been duly considered.

Release notes

None

@guswynn guswynn force-pushed the log-tokio branch 2 times, most recently from a5f6a60 to 78e1761 Compare May 31, 2022 17:37
@guswynn guswynn requested a review from benesch May 31, 2022 17:45
@guswynn guswynn changed the title log chosen tokio-console port in orchestrator-tracing tracing: log chosen tokio-console port in orchestrator-tracing May 31, 2022
Comment thread src/materialized/src/lib.rs Outdated
Comment thread src/orchestrator-tracing/src/lib.rs Outdated
@guswynn

guswynn commented Jun 1, 2022

Copy link
Copy Markdown
Contributor Author

@benesch now that the log-lines have the full names, this is much simpler now!

@benesch

benesch commented Jun 1, 2022

Copy link
Copy Markdown
Contributor

Looks beautiful!

@guswynn guswynn enabled auto-merge (rebase) June 1, 2022 17:31
@guswynn guswynn merged commit 37839d5 into MaterializeInc:main Jun 1, 2022
@guswynn guswynn deleted the log-tokio branch June 2, 2022 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants