-
Notifications
You must be signed in to change notification settings - Fork 0
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
Metadata Manager Depoyment #199
Conversation
williamputraintan
commented
Apr 8, 2024
- Initial deployment of the MetadataManager
- Convert MetadataManager away from EdgeDb to Django
- Readme update
You'r fast! 🙀 .. will review in tick, after lunch! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
lib/workload/stateful/filemanager/ | ||
lib/workload/stateless/ | ||
lib/workload/stateless/filemanager/ | ||
lib/workload/stateless/sequence_run_manager/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hehe. 😬 I will prettier srm in next round.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I like this folder structure.
|
||
-r requirements-slim.txt | ||
|
||
pandas==2.2.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, ho ho. Kung fu panda.!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@williamputraintan I dare you: https:/pola-rs/polars
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Polar (bear) is interesting indeed. One for backlog explore/learning.
The application is configured with [django-simple-history](https://django-simple-history.readthedocs.io/en/latest/) | ||
so that each update to a particular record will have an audit trail. This extension will create an additional table | ||
one-to-one with the original table with `historical` prefixing of the table name. The historical table will have | ||
additional attributes that would indicate addition/update/deletion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done, Will.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for all the work!