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

[incubator-kie-issues-973] Document job service in compact architecture #2001

Merged
merged 2 commits into from
Mar 7, 2024

Conversation

elguardian
Copy link
Contributor

@elguardian elguardian commented Feb 26, 2024

Copy link
Contributor

@martinweiler martinweiler left a comment

Choose a reason for hiding this comment

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

Thanks @elguardian, I only added some minor formatting suggestions.


# Jobs

job service constitutes a subsystem within kogito workflow domain. This component is responsible for scheduling jobs. In the case of workflow this module takes care of timers such from boundary events, SLA, throw events relevant to timers. Also is used for things like human tasks notifications.
Copy link
Contributor

Choose a reason for hiding this comment

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

Job service represents

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done




kogito.persistence.type=jdbc
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd suggest to wrap this in a code block for better readability.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Note: you need to us managed dependencies, otherwise you need to specify the version of the dependencies.
Note: in case you are using other service like MongoDB, check you need to configure the storage with the proper properties. The components uses the common quarkus
Note: in the case of ddl scripts for pgsql you can find it in '''jobs-service-postgresql-common/src/main/resources/db/jobs-service'''

Copy link
Contributor

Choose a reason for hiding this comment

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

The individual notes all get rendered in a single line, so please add at least a blank line in between to avoid this. Or perhaps make it > Note: ... to use quote style.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done



kogito.jobs-service.url=http://localhost:8580

Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above, wrap in a code block.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


<dependency>
<groupId>org.kie.kogito</groupId>
<artifactId>kogito-addons-quarkus-jobs-management</artifactId>
Copy link
Contributor

Choose a reason for hiding this comment

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

Will be renamed to org.kie:kogito-addons-quarkus-jobs-management

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@elguardian elguardian merged commit af326c7 into apache:main Mar 7, 2024
4 of 5 checks passed
rgdoliveira pushed a commit to rgdoliveira/kogito-apps that referenced this pull request Mar 11, 2024
…re (apache#2001)

* [incubator-kie-issues-973] Document job service in compact architecture

* fix comments
pefernan pushed a commit to pefernan/kogito-apps that referenced this pull request Mar 11, 2024
…re (apache#2001)

* [incubator-kie-issues-973] Document job service in compact architecture

* fix comments
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.

Document job service in compact architecture
3 participants