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

Rebrand, refactor, and rewrite to v2 #31

Merged
merged 28 commits into from
Mar 14, 2024
Merged

Rebrand, refactor, and rewrite to v2 #31

merged 28 commits into from
Mar 14, 2024

Conversation

knadh
Copy link
Collaborator

@knadh knadh commented Sep 2, 2023

  • Rebrand sql-jobber to the new name DungBeetle.
  • Significant rewrite and cleanup of the old system.
  • Breaking changes in config variables. Functionality remains the same.

@knadh knadh mentioned this pull request Sep 22, 2023
knadh and others added 26 commits March 13, 2024 14:30
- Create new package `core` and move all core functions there out of
  HTTP handlers.
- De-clutter HTTP handlers to only do validation and call `core`.
- Neatly abstract task loading, queue init and start etc. into `core`.
- Add `init` functions in cmd to de-clutter and simplify `main`.
- Remove hardcoded Machinery references into a generic "queue" namespace
  inside `core`.
- Change "machinery" key in config to "job_queue"

And a lot more ...
* fix: missing error checks and state inconsistencies

* fix: minor lint changes and remove unused constants

* fix: query mapping bug

* refactor: replace default logger with slog

* fix: read config.toml in init

* refactor: use chi middleware for debug logs

* tests: add slow query test case

* chore: add sample config file

* fix: bump tasqueue and fix job result count

---------

Co-authored-by: Lakshay Kalbhor <lakshay.kalbhor@zerodha.com>
kalbhor and others added 2 commits March 13, 2024 14:44
* Refactor docs and branding with the new name, DungBeetle.
* chore: update workflow test file

---------

Co-authored-by: Kailash Nadh <kailash@nadh.in>
Co-authored-by: Lakshay Kalbhor <lakshay.kalbhor@zerodha.com>
@knadh knadh merged commit e8576b2 into master Mar 14, 2024
1 check passed
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