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

feat: Label metrics from live run with "live_run_id" #1943

Merged
merged 7 commits into from
Aug 30, 2023
Merged

feat: Label metrics from live run with "live_run_id" #1943

merged 7 commits into from
Aug 30, 2023

Conversation

chubei
Copy link
Contributor

@chubei chubei commented Aug 30, 2023

Closes #1940

Also the backend part of #1915

@Jesse-Bakker
Copy link
Contributor

Can we make a metrics Recorder that drives the progress bars instead of having to update the progress bars from components like cache_builder?

@chubei
Copy link
Contributor Author

chubei commented Aug 30, 2023

I don't want to keep track of a HashMap<String, ProgressBar> because that'll requrie interior mutability. Any way around this?

@chubei
Copy link
Contributor Author

chubei commented Aug 30, 2023

Oh I misunderstood. You meant to drive progress bar from metrics. I believe that's out of the scope of this PR.

Copy link
Contributor

@Jesse-Bakker Jesse-Bakker left a comment

Choose a reason for hiding this comment

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

Small style nit, otherwise LGTM!

dozer-cli/src/live/progress.rs Outdated Show resolved Hide resolved
@chubei chubei added this pull request to the merge queue Aug 30, 2023
@snork-alt
Copy link
Contributor

@chubei Why does this require doc update ?

@chubei
Copy link
Contributor Author

chubei commented Aug 30, 2023

Sorry bogus auto labelling. Removing now.

Merged via the queue into getdozer:main with commit 97ff02d Aug 30, 2023
7 checks passed
@chubei chubei deleted the feat/label branch August 30, 2023 11:01
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.

After Start/Stop in Dozer Live data keeps accumulating in the progress
3 participants