Skip to content

incubator-kie-issues#2307 data-isolation by process id and version pair#2337

Merged
pefernan merged 20 commits into
apache:mainfrom
jstastny-cz:incubator-kie-issues#2307-versions
Jul 3, 2026
Merged

incubator-kie-issues#2307 data-isolation by process id and version pair#2337
pefernan merged 20 commits into
apache:mainfrom
jstastny-cz:incubator-kie-issues#2307-versions

Conversation

@jstastny-cz

@jstastny-cz jstastny-cz commented May 22, 2026

Copy link
Copy Markdown
Contributor

Fixes:

Part of ensemble:

Broad changes to accommodate for possible filtering of DB entries by process id and version pairs. Important aspect is that the engine was currently not properly populating the version information throughout the internal models, so there are excessive changes related to this, while the actual data-isolation filtering is small footprint.

Moreover, for sake of properly isolate items belonging to reusable subprocesses, id and version information needs to be exposed also for root process, so via root process id and root process version.

Areas affected:

  • Data Events - data events were refactored to properly set the new fields, instead of adding yet another constructor with way too many parameters, I implemented builder pattern for the data event objects. This should be now the encouraged way, I have marked the existing multi-parameter constructors as deprecated.
  • Database queries - actual database queries are being composed dynamically and depending on local processes, appropriate filtering is appended and parameters bound.
  • Conditional enabling of this feature by passing or not Processes bean to Repository class implementations.

Components isolated:

  • data-index storage
  • embedded jobs addon storage

Components planned in a follow-up PR (for sake of reducing size):

  • data-audit storage

@jstastny-cz jstastny-cz force-pushed the incubator-kie-issues#2307-versions branch from 26075be to 8e261e9 Compare June 25, 2026 14:34
@jstastny-cz jstastny-cz marked this pull request as ready for review June 26, 2026 07:56
@jstastny-cz jstastny-cz changed the title incubator-kie-issues#2307 considering versions incubator-kie-issues#2307 data-isolation by process id and version pair Jun 26, 2026
@jstastny-cz jstastny-cz force-pushed the incubator-kie-issues#2307-versions branch from b2d79bf to 4cd3f3c Compare July 1, 2026 08:38

@martinweiler martinweiler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@jstastny-cz my apologies that I caused the conflict in JPAQuery by merging PR#2328. Once you have resolved this, we should be good with this PR as well. Great work!

@jstastny-cz jstastny-cz force-pushed the incubator-kie-issues#2307-versions branch from 56dc096 to e182ba7 Compare July 3, 2026 09:02
@pefernan pefernan merged commit eb2e3ff into apache:main Jul 3, 2026
5 checks 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.

3 participants