Skip to content

Support ConnectionContextAware for DistSQLQueryExecutor#37469

Merged
sandynz merged 4 commits into
apache:masterfrom
sandynz:dev-1
Dec 23, 2025
Merged

Support ConnectionContextAware for DistSQLQueryExecutor#37469
sandynz merged 4 commits into
apache:masterfrom
sandynz:dev-1

Conversation

@sandynz
Copy link
Copy Markdown
Contributor

@sandynz sandynz commented Dec 23, 2025

Changes proposed in this pull request:

  • Add QueryContext param in DistSQLUpdateProxyBackendHandler constructor. Used to build DistSQLConnectionContext
  • Refactor DistSQLUpdateExecuteEngine constructor, replace ConnectionContext to DistSQLConnectionContext. Use unified DistSQLConnectionContext with DistSQLQueryExecuteEngine
  • Refactor DistSQLExecutorAwareSetter set method, remove ConnectionContext param. Used unified DistSQLConnectionContext, and get ConnectionContext from DistSQLConnectionContext

Before committing this PR, I'm sure that I have checked the following options:

  • My code follows the code of conduct of this project.
  • I have self-reviewed the commit code.
  • I have (or in comment I request) added corresponding labels for the pull request.
  • I have passed maven check locally : ./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.
  • I have made corresponding changes to the documentation.
  • I have added corresponding unit tests for my changes.
  • I have updated the Release Notes of the current development version. For more details, see Update Release Note

@sandynz sandynz merged commit 0f04f33 into apache:master Dec 23, 2025
145 checks passed
@sandynz sandynz deleted the dev-1 branch December 23, 2025 08:05
makssent pushed a commit to makssent/shardingsphere that referenced this pull request Apr 9, 2026
…ngsphere/master

* remotes/origin/master:
  Support ConnectionContextAware for DistSQLQueryExecutor (apache#37469)
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