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

[NAE-1988] Extend Id of processes resource for better resolution #262

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

tuplle
Copy link
Member

@tuplle tuplle commented Jun 28, 2024

Description

Implements new id object ProcessResourceId that represents the id of an object that belongs to some process. The id is defined with two parts with the symbol '-' as a separator. The first part, prefix, is encoded Process (PetriNet) ObjectId named process short. The second part is an ObjectId that MongoDB generates and thus it is backwards compatible.

This definition of an id helps to identify to what process an object belongs to quickly. It is helpful in the process-resolving for NAE cluster.

Implements NAE-1988

Dependencies

There are no dependencies on other PR

How Has Been This Tested?

Manually tested and current tests refactored to match new id.

Test Configuration

Name Tested on
OS Windows 11
Runtime Java OpenJDK 21
Dependency Manager Maven 3
Framework version Spring Boot 3.2.5
Run parameters
Other configuration

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes have been checked, personally or remotely, with @machacjozef, @renczesstefan
  • I have commented my code, particularly in hard-to-understand areas
  • I have resolved all conflicts with the target branch of the PR
  • I have updated and synced my code with the target branch
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes:
    • Lint test
    • Unit tests
    • Integration tests
  • I have checked my contribution with code analysis tools:
  • I have made corresponding changes to the documentation:
    • Developer documentation
    • User Guides
    • Migration Guides

- add process resource id
- refactor task, case, process role to new id object
@tuplle tuplle added improvement A change that improves on an existing feature critical A change that is critical to a release and must not be omitted labels Jun 28, 2024
@tuplle tuplle self-assigned this Jun 28, 2024
@tuplle tuplle merged commit 4a163f0 into release/7.0.0 Jun 28, 2024
4 of 6 checks passed
@tuplle tuplle deleted the NAE-1988 branch June 28, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
critical A change that is critical to a release and must not be omitted improvement A change that improves on an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant