-
Notifications
You must be signed in to change notification settings - Fork 5
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
Improve performance sync relations #107
Conversation
1c7332d
to
dbe3da6
Compare
Codecov Report
@@ Coverage Diff @@
## master #107 +/- ##
==========================================
- Coverage 96.17% 96.17% -0.01%
==========================================
Files 40 40
Lines 1832 1831 -1
==========================================
- Hits 1762 1761 -1
Misses 70 70
Continue to review full report at Codecov.
|
dbe3da6
to
68b9d53
Compare
geostore/models.py
Outdated
|
||
logger.info(f"{to_delete.count()} element(s) to delete") | ||
# logger.info(f"{to_delete.count()} element(s) to delete") |
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.
Do you need to keep this comments
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.
No
geostore/models.py
Outdated
logger.info(f"{len(qs)} element(s) to add") | ||
qs = Feature.objects.filter(id__in=id_list).exclude(pk__in=self.relations_as_origin.filter(relation=rel) | ||
.values_list('destination_id', flat=True)) | ||
# logger.info(f"{len(qs)} element(s) to add") |
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.
And this one?
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.
No
68b9d53
to
d4b4b5a
Compare
No description provided.