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

refactor(consumption_metrics): split #5326

Merged
merged 4 commits into from
Sep 16, 2023

Conversation

koivunej
Copy link
Member

@koivunej koivunej commented Sep 16, 2023

Split off from #5297. Builds upon #5325, should contain only the splitting. Next up: #5327.

@koivunej koivunej requested review from a team as code owners September 16, 2023 11:39
@koivunej koivunej requested review from save-buffer and problame and removed request for a team September 16, 2023 11:39
@koivunej koivunej marked this pull request as draft September 16, 2023 11:39
@koivunej koivunej force-pushed the refactor_consumption_metrics_test_7 branch from d1d23bb to 7f8b78e Compare September 16, 2023 11:43
@koivunej koivunej force-pushed the refactor_consumption_metrics_test_8 branch from bdfe3d6 to d1d23bb Compare September 16, 2023 11:44
@koivunej koivunej changed the title refactor(consumption_metrics): post split cleanups, test support refactor(consumption_metrics): split Sep 16, 2023
@github-actions
Copy link

github-actions bot commented Sep 16, 2023

2484 tests run: 2363 passed, 0 failed, 121 skipped (full report)


Flaky tests (2)

Postgres 15

  • test_timeline_status[False]: release

Postgres 14

  • test_get_tenant_size_with_multiple_branches: debug

Code coverage (full report)

  • functions: 53.0% (7744 of 14615 functions)
  • lines: 81.0% (45227 of 55832 lines)

The comment gets automatically updated with the latest test results
b92335c at 2023-09-16T15:49:18.462Z :recycle:

@koivunej koivunej marked this pull request as ready for review September 16, 2023 12:25
Base automatically changed from refactor_consumption_metrics_test_7 to main September 16, 2023 15:08
koivunej added a commit that referenced this pull request Sep 16, 2023
Cleanups in preparation to splitting the consumption_metrics.rs in
#5326.

Split off from #5297.
@koivunej koivunej force-pushed the refactor_consumption_metrics_test_8 branch from d1d23bb to b92335c Compare September 16, 2023 15:10
@koivunej koivunej merged commit e62ab17 into main Sep 16, 2023
@koivunej koivunej deleted the refactor_consumption_metrics_test_8 branch September 16, 2023 15:45
koivunej added a commit that referenced this pull request Sep 18, 2023
Split off from #5297. Builds upon #5326. Handles original review
comments which I did not move to earlier split PRs. Completes test
support for verifying events by notifying of the last batch of events.
Adds cleaning up of tempfiles left because of an unlucky shutdown or
SIGKILL.

Finally closes #5175.

Co-authored-by: Arpad Müller <[email protected]>
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