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

Main #8

Open
wants to merge 11 commits into
base: ARO-1208
Choose a base branch
from
Open

Main #8

wants to merge 11 commits into from

Conversation

tuurek
Copy link
Contributor

@tuurek tuurek commented Jun 6, 2023

Merging changes for limiting the amount of changes read from the db at one time.

matti-lamppu and others added 11 commits May 24, 2023 15:59
Merging changes from Matti Lamppu

* Add .gitattributes

This file helps convert line endings correctly for windows users

* Add requirements.txt to .gitignore

* Add docker-compose setup for testing

* Add support for django_auditlog

- Support of django_auditlog is optional and must be enabled with
  an environment variable
- Created a common interface that abstracts the custom database model and
  django_audtilog model
- Added better typing
- Refactored and formatted code
- Added tests for LogEnties
- Added and improved on documentation

* Add support for old log entries without is_sent in additional_data

* Simplify django_auditlog schema example

* Fix docker compose testing setup and add a second configuration

Configurations have to be selected using docker profiles, or with the
included makefile scrips

* Remove unused import

* Change USE_DJANGO_AUDITLOG setting to an enum

This allows adding more configurations more easily in the future.
BATCH_LIMIT controls how many logs are fetched during a single job
CHUNK_SIZE controls how many logs are put into memory at once
This reverts commit d440e63.
The solution required read access to logs for log writer, reverted.
…12)

* Added missing import Generator

* Fix to be able to use select_for_update together with select_related

* eq self add only to select_related

* removed select_for_update()

* removed all instances of select_for_update in task py

* testing of self

---------

Co-authored-by: Darius-Seputis <[email protected]>
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.

4 participants