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

Update cron documentation to include Spring formatting #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

essential-randomness
Copy link

I completely removed the Wikipedia link as that specifically discusses the Unix format and will cause confusion. Instead, I swapped it with a link to the Spring documentation.

…and clarify the format of the cron expression.
Copy link
Member

@lbroudoux lbroudoux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please find another link 😉

@@ -347,7 +347,7 @@ <h3 id="creating-a-new-scheduled-import">Creating a new scheduled import</h3>
<p><img src="/images/importer-step3.png" alt="importer-step3"></p>
<p>On first time creation the Job is automatically <code>Scanned</code> and <code>Imported</code>.</p>
<h3 id="configure-scheduling-interval">Configure scheduling interval</h3>
<p>The scheduling interval can be globally configured for all the Jobs. It is a global setting and not a per-Job one. This is achieved through the <code>services.update.interval</code> property in the <code>application.properties</code> configuration file that takes the value of <code>SERVICES_UPDATE_INTERVAL</code> environment variable. The value should be set to a valid <a href="https://en.wikipedia.org/wiki/Cron#CRON_expression">CRON expression</a> ; default is every 2 hours.</p>
<p>The scheduling interval can be globally configured for all the Jobs. It is a global setting and not a per-Job one. This is achieved through the <code>services.update.interval</code> property in the <code>application.properties</code> configuration file that takes the value of <code>SERVICES_UPDATE_INTERVAL</code> environment variable. The value should be set to a valid <a href="https://docs.oracle.com/cd/E12058_01/doc/doc.1014/e12030/cron_expressions.htm">CRON expression in Spring format</a> ; default is every 2 hours.</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a big fan of having a link to Oracle documentation... you were mentioning Spring documentation in your comment. I've found this link to be helpful: https://spring.io/blog/2020/11/10/new-in-spring-5-3-improved-cron-expressions
What do you think?

@lbroudoux
Copy link
Member

Thanks for the PR and sorry for little delay.

Also, sorry to bother you but this is the wrong repo... We use https://github.com/microcks/microcks.io that is Markdown content to generate and commit automatically on this repo. Would you mind reapplying to this repo? Namely on the https://github.com/microcks/microcks.io/blob/master/content/documentation/using/importers.md file.

I know it could be boring and I can do for you if you prefer.

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.

2 participants