-
Notifications
You must be signed in to change notification settings - Fork 2
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
dev #366
base: master
Are you sure you want to change the base?
dev #366
Conversation
[e37524]: Handle scenario when no disaggregation exists for ReportsAppliedindicator
…d of last_run==now
FIX: increase section_name field length
HOTFIX: last_modified _field flag
…logic Add changes for pdindicator fix and loader delta logic
…hub.com/unicef/etools-datamart into fix_pdindicator_and_loader_delta_logic2
…logic2 Fix versioning info
Add celery task failure capture mechanism
…issmatch Workaround pdindicator target fieldmissmatch
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.
@@ -20,3 +20,10 @@ def healthcheck(): | |||
# @app.on_after_configure.connect | |||
# def setup_periodic_tasks(sender, **kwargs): | |||
# sender.add_periodic_task(60.0, healthcheck.s(), name='healthcheck') | |||
|
|||
|
|||
@app.task() |
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.
please remove
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.
Removed in PR for [ch38092] - still to be merged to develop
|
||
def process_country(self): | ||
batch_size = settings.RESULTSET_BATCH_SIZE | ||
logger.debug(f"Batch size:{batch_size}") | ||
|
||
qs = self.get_queryset().exclude(intervention__isnull=True) |
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.
add this back in.
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.
PartnersInterventionbudget.objects.exclude(intervention__isnull=True) is already there.
[ch38120]: Export usage log should include user as well
[ch38091] delete records at country level in the target table
[ch38072][ch38092] Loader country level err report
[ch37970] add additional filter for EtoolsUser based on groups values
[ch33585] Fix intermittent disappearance of country and record count information in RUNNING status for ETL loaders
No description provided.