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

Fix for (some) pdf reports not being send #1089

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

kosarko
Copy link
Member

@kosarko kosarko commented Dec 4, 2023

With the default config the lr.email.burst is 20. If clarin-dspace needs to send more than lr.email.burst mails, some of them get queued. The piwik-report-generator is a short lived process, it does not wait for the queue to be empty and exits. Hence any email in the queue is actually not sent.

This fix just changes lr.email.burst when calling make generate_piwik_reports. System properties have a rather low priority. If you have lr.email.burst in config/modules/*.cfg or config/dspace.cfg this fix will not work.

With the default config the `lr.email.burst` is 20.  If clarin-dspace needs to send
more than `lr.email.burst` mails, some of them get queued.
The piwik-report-generator is a short lived process, it does not wait for the queue
to be empty and exits. Hence any email in the queue is actually not sent.

This fix just changes `lr.email.burst` when calling `make generate_piwik_reports`.
System properties have a rather low priority. If you have
`lr.email.burst` in `config/modules/*.cfg` or `config/dspace.cfg` this
fix will not work.
@kosarko kosarko merged commit 63511f3 into ufal:clarin-dev Dec 4, 2023
1 check passed
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.

1 participant