-
Notifications
You must be signed in to change notification settings - Fork 50
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
feat: AAP-30754 - Add analytics collector to EDA #1048
Conversation
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.
@hsong-rh What if we used the ORM to generate all the queries? This approach would improve maintainability and help prevent potential issues when changes are made to the model later on. Just make sure that the datetime values are properly formatted.
Even I would suggest to explore https://palewi.re/docs/django-postgres-copy/ which adds COPY support for django. |
714dc31
to
321a0b1
Compare
Good suggestions, will work on it in the future. |
36255e1
to
2cd7ae5
Compare
6f53b3f
to
d17bbc4
Compare
31330b0
to
79e132b
Compare
d43d989
to
bd0f61d
Compare
Signed-off-by: Alex <aizquier@redhat.com> Co-authored-by: Bill Wei <bilwei@redhat.com> Co-authored-by: Joe Shimkus <35382397+jshimkus-rh@users.noreply.github.com> Co-authored-by: Elijah DeLee <kdelee@redhat.com>
|
moved to #1065 |
https://issues.redhat.com/browse/AAP-30754
depends: #1049