Skip to content

[fix][CGS][entrance] fix dependency injection failure in entrance configuration#980

Merged
casionone merged 2 commits intodev-2.0.0from
dev-2.0.0-bug-fix
Apr 8, 2026
Merged

[fix][CGS][entrance] fix dependency injection failure in entrance configuration#980
casionone merged 2 commits intodev-2.0.0from
dev-2.0.0-bug-fix

Conversation

@v-kkhuang
Copy link
Copy Markdown

What is the purpose of the change

Background/Problem:
Dependency injection failure occurs in certain components during service startup, particularly in Entrance configuration and Flink plugin. This causes services to fail to start properly and affects system stability.

Purpose of Change:
To address this issue, this PR fixes the dependency injection problems in Entrance configuration and improves exception comments for better debugging and troubleshooting.

Value/Impact:
After the change, services can start properly without dependency injection errors, improving system reliability and maintainability.

Related issues/PRs

Related issues: close #979
Related pr:none

Brief change log

  • Fix dependency injection failure in Entrance configuration
  • Fix dependency injection issues in Flink plugin
  • Improve exception comments for better debugging

Checklist

  • I have read the Contributing Guidelines on pull requests.
  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible
  • If this is a code change: I have written unit tests to fully verify the new behavior.

@v-kkhuang v-kkhuang added the bug Something isn't working label Apr 8, 2026
Copy link
Copy Markdown

@casionone casionone left a comment

Choose a reason for hiding this comment

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

LGTM.

@casionone casionone merged commit 614b0f3 into dev-2.0.0 Apr 8, 2026
16 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants