Skip to content

Polish unused code follow-up - #15441

Merged
KomachiSion merged 1 commit into
alibaba:developfrom
KomachiSion:codex/remove-unused-pipeline-sql
Jun 26, 2026
Merged

KomachiSion merged 1 commit into
alibaba:developfrom
KomachiSion:codex/remove-unused-pipeline-sql

Conversation

@KomachiSion

Copy link
Copy Markdown
Collaborator

Summary

  • Remove a leftover unused SQL constant from PipelineExecutionRepositoryImpl.
  • Mark IConfigRequest#getConfigContext as deprecated to match the existing IConfigResponse compatibility marker.

Testing

  • mvn -pl ai,api spotless:apply spotless:check -DskipTests
  • mvn -pl ai,api -am -DskipTests compile
  • mvn -pl ai -am -Dtest=PipelineExecutionRepositoryImplTest -DfailIfNoTests=false -Dsurefire.failIfNoSpecifiedTests=false test

Remove the leftover unused pipeline resource SQL constant.

Mark IConfigRequest#getConfigContext as deprecated for compatibility.

Assisted-by: Claude Code
@github-actions

Copy link
Copy Markdown

Thanks for your this PR. 🙏
Please check again for your PR changes whether contains any usage/api/configuration change such as Add new API , Add new configuration, Change default value of configuration.
If so, please add or update documents(markdown type) in docs/next/ for repository nacos-group/nacos-group.github.io


感谢您提交的PR。 🙏
请再次查看您的PR内容,确认是否包含任何使用方式/API/配置参数的变更,如:新增API、新增配置参数、修改默认配置等操作。
如果是,请确保在提交之前,在仓库nacos-group/nacos-group.github.io中的docs/next/目录下添加或更新文档(markdown格式)。

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@KomachiSion
KomachiSion merged commit 8847043 into alibaba:develop Jun 26, 2026
7 checks passed
@KomachiSion KomachiSion added this to the 3.3.0 milestone Jun 26, 2026
@KomachiSion
KomachiSion deleted the codex/remove-unused-pipeline-sql branch June 26, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants