-
Notifications
You must be signed in to change notification settings - Fork 3
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
XRDCAT-12: Improve Company and Organization handling processes #28
Conversation
Job Summary for GradleX-Road-Catalog tests :: BuildAndTest
|
Job Summary for GradleX-Road-Catalog tests :: BuildAndTest
|
| `xroad-catalog.fetch-external-time-after-hour` | 3 | A parameter for setting the start of time interval during which the X-Road Catalog Collector should try to fetch data from the companies API continuously (this parameter will be ignored if the parameter `xroad-catalog.fetch-companies-run-unlimited` is set to `true`), e.g. value `18` means starting from `18:00`. | | ||
| `xroad-catalog.fetch-external-time-before-hour` | 4 | A parameter for setting the end of time interval during which the X-Road Catalog Collector should try to fetch data from the companies API continuously (this parameter will be ignored if the parameter `xroad-catalog.fetch-companies-run-unlimited` is set to `true`), e.g. value `23` means ending at `23:00`. | | ||
| `xroad-catalog.fetch-external-interval-min` | 20 | A parameter for setting the amount of time in minutes after which the X-Road Catalog Collector should start re-fetching data from the external API, e.g. value `20` means `every 20 minutes`. This works together with the following two parameters to determine how often data is checked for staleness and updated in a batch. | | ||
| `xroad-catalog.fetch-external-limit` | 500 | A parameter for setting the maximum amount of Members that can be fetched from the external API in one run, e.g. value `500` means `500 members`. | |
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.
For clarity, it would be good to specify if the limit is applied per API or for both APIs together, e.g., 500 Companies + 500 Organizations OR Companies + Organizations = 500.
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.
Updated the wording to make what happens clearer.
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.
Thanks! Now it's clear.
...-collector/src/test/java/fi/vrk/xroad/catalog/collector/tasks/FetchOrganizationTaskTest.java
Show resolved
Hide resolved
...log-collector/src/test/java/fi/vrk/xroad/catalog/collector/tasks/FetchCompaniesTaskTest.java
Show resolved
Hide resolved
Quality Gate failedFailed conditions |
Job Summary for GradleX-Road-Catalog tests :: BuildAndTest
|
No description provided.