Skip to content

[feature][COM][commons] optimize linkis logging#5378

Merged
casionone merged 2 commits intoapache:dev-1.9.0from
aiceflower:relaese-1.9.0-manager-code-optimize
Mar 26, 2026
Merged

[feature][COM][commons] optimize linkis logging#5378
casionone merged 2 commits intoapache:dev-1.9.0from
aiceflower:relaese-1.9.0-manager-code-optimize

Conversation

@aiceflower
Copy link
Member

@aiceflower aiceflower commented Mar 26, 2026

Search before asking

  • I had searched in the issues and found no similar feature requirement.

Problem Description

Engine creation and reuse service performance needs optimization.

Description

Optimize engine creation and reuse service for better performance and reduced latency.

Use case

Improve engine creation and reuse performance in production environments with high concurrency.

Solutions

Optimized DefaultEngineCreateService and DefaultEngineReuseService for better performance.

Anything else

None.

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

- 在 AMUtils 中引入复用线程池,将线程数设置为 5 来提升指标更新效率
- 修改 DefaultEngineAskEngineService 中的条件逻辑结构,优化引擎复用节点的处理流程
- 将 DefaultEngineCreateService 中的日志级别从 info 提升到 warn 并添加异常堆栈信息
- 在 DefaultEngineCreateService 和 DefaultEngineReuseService 中添加空值检查来避免 NPE
- 使用 tryCatch 包装指标更新逻辑以提高系统稳定性
- 从引擎复用过滤条件中移除节点状态解锁检查以提升复用率
@aiceflower aiceflower changed the title [perf][CGS][manager] optimize engine creation and reuse service performance [feature][COM][commons] optimize Linkis logging Mar 26, 2026
@aiceflower aiceflower changed the title [feature][COM][commons] optimize Linkis logging [feature][com][commons] optimize linkis logging Mar 26, 2026
@aiceflower aiceflower changed the title [feature][com][commons] optimize linkis logging [feature][COM][commons] optimize linkis logging Mar 26, 2026
Copy link
Contributor

@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 7b130d9 into apache:dev-1.9.0 Mar 26, 2026
5 of 11 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