chore: pkg/db cleanup #798

Closed
opened 2026-03-28 04:28:39 +00:00 by mfreeman451 · 1 comment
Owner

Imported from GitHub.

Original GitHub issue: #2399
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/issues/2399
Original created: 2026-01-20T03:49:49Z


seems like pkg/db has a ton of stuff in there that we shouldn't need anymore, I believe the only package still using pkg/db is db-event-writer. Having anything write to the database that is not in elixir and can't share types with is a pain in the butt/tech debt, but having all of this extra code to handle things that we used to do in the older golang services is even worse. Need to cleanup this, it should be lean and mean and only provide things that we can verify that db-event-writer needs.

Imported from GitHub. Original GitHub issue: #2399 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/issues/2399 Original created: 2026-01-20T03:49:49Z --- seems like `pkg/db` has a ton of stuff in there that we shouldn't need anymore, I believe the only package still using pkg/db is `db-event-writer`. Having anything write to the database that is not in elixir and can't share types with is a pain in the butt/tech debt, but having all of this extra code to handle things that we used to do in the older golang services is even worse. Need to cleanup this, it should be lean and mean and only provide things that we can verify that `db-event-writer` needs.
Author
Owner

Imported GitHub comment.

Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/issues/2399#issuecomment-3771016256
Original created: 2026-01-20T04:44:09Z


closing as completed

Imported GitHub comment. Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/issues/2399#issuecomment-3771016256 Original created: 2026-01-20T04:44:09Z --- closing as completed
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
carverauto/serviceradar#798
No description provided.