Skip to content

Changed ServiceGroup: from GenerationType.AUTO to GenerationType.IDEN…#329

Open
stanleykc wants to merge 1 commit into
mainfrom
bug/issue-325-inconsistent-id-gen
Open

Changed ServiceGroup: from GenerationType.AUTO to GenerationType.IDEN…#329
stanleykc wants to merge 1 commit into
mainfrom
bug/issue-325-inconsistent-id-gen

Conversation

@stanleykc

@stanleykc stanleykc commented Jan 7, 2026

Copy link
Copy Markdown
Member

Summary

This PR fixes the inconsistent ID generation issue.

This issue was uncovered when creating a new service group using the existing Service Request Definition Editor. It is a backend issue that will impact a new editor as well.

This makes ServiceGroup consistent with all other entities in the codebase and uses MySQL's native AUTO_INCREMENT instead of the hibernate_sequence table, which prevents the duplicate key errors when creating new service groups.

Fixes #325

Changes

  • Updated ID generation logic
  • Added validation tests

…TITY. This makes ServiceGroup consistent with all other entities in the codebase and uses MySQL's native AUTO_INCREMENT instead of the hibernate_sequence table, which prevents the duplicate key errors when creating new service groups.

Signed-off-by: Kevin Stanley <stanleyk@objectcomputing.com>
@stanleykc stanleykc requested a review from jjelliott January 7, 2026 21:36

@jjelliott jjelliott 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.

Does this need an accompanying schema update?

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.

ServiceGroup uses inconsistent ID generation strategy causing duplicate key errors

3 participants