Skip to content

Conversation

@RaigorJiang
Copy link
Contributor

Fixes #36379.

Changes proposed in this pull request:

  • Add temporary config key instance-connection-enabled, the default value is false
  • Data sources will be aggregated only when instance-connection-enabled is true and the database type is instance connection available.
  • In CatalogSwitchableDataSource, before set catalog, check the current catalog is different or not

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

@jiangML jiangML merged commit 5006a9f into apache:master Jan 9, 2026
145 checks passed
@RaigorJiang RaigorJiang deleted the instance-connection branch January 9, 2026 09:39
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.

Aggregate data sources increase shardingproxy latecy and reduced performance

2 participants