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.

Sign up for GitHub

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

Jump to bottom

Allow KieRuntimeBuilder to also create and provide StatelessKieSession #1492

Closed
mariofusco opened this issue Sep 23, 2024 · 1 comment · Fixed by apache/incubator-kie-drools#6103
Closed

Allow KieRuntimeBuilder to also create and provide StatelessKieSession #1492

mariofusco opened this issue Sep 23, 2024 · 1 comment · Fixed by apache/incubator-kie-drools#6103
Assignees
@mariofusco
Labels
area:engine Related to the runtime engines type:enhancement Something that already exists needs to be improved

Comments

@mariofusco
Copy link

At the moment the KieRuntimeBuilder only allow to retrieve KieSession s, but not StatelessKieSession s. Even worse when a KieSession configured as stateless is required through the current API it creates a stateful one anyway, making things even more confused.

Finally the impossibility of creating a StatelessKieSession from KieRuntimeBuilder is also the indirect cause of a memory leak with the Kogito integration as reported here.
@mariofusco mariofusco self-assigned this Sep 23, 2024
@mariofusco mariofusco mentioned this issue Sep 25, 2024
Merged
@yesamer yesamer added type:enhancement Something that already exists needs to be improved area:engine Related to the runtime engines labels Sep 26, 2024
@mariofusco
Copy link
Author

Fixed by apache/incubator-kie-drools#6103 and apache/incubator-kie-kogito-runtimes#3681

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mariofusco mariofusco

Labels
area:engine Related to the runtime engines type:enhancement Something that already exists needs to be improved
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[KIE-1492] Allow KieRuntimeBuilder to also create and provide StatelessKieSession mariofusco/drools
2 participants
@mariofusco @yesamer