Skip to content

[BugFix] Fix ECM TooManyResultsException after abnormal restart (#5316) - #5473

Open
zhang-arvin wants to merge 1 commit into
apache:masterfrom
zhang-arvin:fix/5316-ecm-toomanyresults
Open

zhang-arvin wants to merge 1 commit into
apache:masterfrom
zhang-arvin:fix/5316-ecm-toomanyresults

Conversation

@zhang-arvin

Copy link
Copy Markdown

Closes #5316

… restart (apache#5316)

When ECM abnormally shuts down and restarts, duplicate records may exist in the
database causing MyBatis selectOne() to throw TooManyResultsException. This fix
adds LIMIT 1 to single-result queries in NodeManagerMapper to prevent the
exception when multiple records are unexpectedly returned.

Affected queries:
- getNodeInstance: LIMIT 1 added
- getNodeInstanceByTicketId: LIMIT 1 added
- getEMNodeInstanceByEngineNode: LIMIT 1 added
@zhang-arvin

Copy link
Copy Markdown
Author

Base-branch note: the failing checks are not caused by this PR.

All of sql-check, spotless-check, third-party-dependencies-check and build-backend fail on files that this PR does not touch, and they fail identically on the other open PRs from the same branch point. The failures come from master itself:

None of these files appears in this PR's diff, so there is nothing on this PR to fix here; it will go green once master is repaired. This PR stays on the master baseline problem until that lands.

This comment was generated by an AI agent (Hermes Agent).

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.

[Bug][ECM] TooManyResultsException error when querying ECM management after abnormal restart

1 participant