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.

Sign up for GitHub

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

Jump to bottom

Create and clean up kafka topics manually to reduce memory burden for Adala server #107

Merged
merged 15 commits into from May 21, 2024
Merged

Create and clean up kafka topics manually to reduce memory burden for Adala server #107

merged 15 commits into from May 21, 2024

Conversation

matt-bernstein
Copy link
Contributor

No description provided.

@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/7fafaf80164b9a0a8dbd6a8ae1a7125c4328ee5f is created.

@codecov-commenter
Copy link

codecov-commenter commented May 8, 2024 •
edited

Codecov Report

Attention: Patch coverage is 0% with 27 lines in your changes are missing coverage. Please review.

Project coverage is 44.46%. Comparing base (b20edc1) to head (d9a1ec5).

Files Patch % Lines
server/utils.py 0.00% 24 Missing ⚠️
server/tasks/process_file.py 0.00% 3 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #107      +/-   ##
==========================================
- Coverage   45.06%   44.46%   -0.60%     
==========================================
  Files          39       39              
  Lines        1782     1806      +24     
==========================================
  Hits          803      803              
- Misses        979     1003      +24

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/129c5bc7a8729dd74b64383b11390c224a201081 is created.

hakan458
hakan458 reviewed May 8, 2024
View reviewed changes
server/utils.py Outdated Show resolved Hide resolved
@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/1c746fad389ccce43816e0be29288e4fb72df292 is created.

@matt-bernstein matt-bernstein force-pushed the fb-dia-1112/cleanup-kafka-topics branch from ac7e310 to 6a04f1e Compare May 8, 2024 20:32
@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/3b00c9e37d9e47f3b30285f1b5bc08b148834b21 is created.

@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/14380a999c9abcee342c2b9f640c7d4936626af1 is created.

@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/3f06167275b421d788aeacb247e9f7c6acc11628 is created.

@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/c227c64baa6666dac855df947bba18a4530459ca is created.

@matt-bernstein matt-bernstein force-pushed the fb-dia-1112/cleanup-kafka-topics branch from 7eb7ee1 to d9a1ec5 Compare May 9, 2024 00:22
@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/e2094004ad3e9f662f712802cce44fac17fe8b02 is created.

@robot-ci-heartex robot-ci-heartex marked this pull request as draft May 9, 2024 11:04
@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/2193a9038b487f0868d6f067720579bbe45f53d5 is created.

pakelley
pakelley reviewed May 16, 2024
View reviewed changes
server/utils.py Outdated Show resolved Hide resolved
@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/b1c6f45809a2360c0966d0e3536ddea55e4d8648 is created.

@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/c23b8f3b35ebef3b631628535cd8c16125bf8ddc is created.

@matt-bernstein matt-bernstein marked this pull request as ready for review May 21, 2024 16:55
@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/9fcff5cdbcfcee52e1705435fe4f8703aa616480 is created.

@matt-bernstein matt-bernstein merged commit d163a69 into master May 21, 2024
24 of 27 checks passed
@matt-bernstein matt-bernstein deleted the fb-dia-1112/cleanup-kafka-topics branch May 21, 2024 20:50
@robot-ci-heartex
Copy link
Collaborator

Commit https://github.com/HumanSignal/infra/commit/7c3beca199ab1c539f3bacd783d81f32cf70b2b0 is created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@forum-hs forum-hs forum-hs left review comments

@pakelley pakelley pakelley approved these changes

@hakan458 hakan458 hakan458 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@matt-bernstein @robot-ci-heartex @codecov-commenter @pakelley @hakan458 @forum-hs