Skip to content

fix: reuse db connection in CONSOLE_DS_KEY #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: dev/4.3.4
Choose a base branch
from
Open

fix: reuse db connection in CONSOLE_DS_KEY #2

wants to merge 3 commits into from

Conversation

@littleniannian
Copy link

@littleniannian littleniannian commented Nov 17, 2025 •
edited
Loading

修复方案

  • register的时候保留注册BACKEND_DATASOURCE
  • 获取数据库PL对象(PLObjectErrMsgUtils)，查看数据库详情(DBSchemaAccessors)，查询当前数据库存在的会话(DBSessionService),负责会话窗口的相关操作(DefaultDBSessionManage),对数据库对象(FUNCTION,PROCEDURE,TYPE)的操作,SQL执行计划(SqlDiagnoseService,OBQueryProfileManager),分区表操作(PartitionPlanService),事务回滚(RollbackGeneratorFactory) 这些本来使用BackendDatasource的地方使用ConsoleDataSource进行复用。
  • 其余的数据归档，OB性能分析，影子表同步因为在页面上做了隐藏所以不做修改，保持原来的样式。

littleniannian
littleniannian commented Nov 17, 2025
View reviewed changes
server/odc-service/src/main/java/com/oceanbase/odc/service/common/util/PLObjectErrMsgUtils.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

获取数据库PL对象的编译错误信息

server/odc-service/src/main/java/com/oceanbase/odc/service/db/browser/DBSchemaAccessors.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

下拉数据库,查看数据库详情信息

server/odc-service/src/main/java/com/oceanbase/odc/service/db/session/DBSessionService.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

查询当前数据库存在的会话

...r/odc-service/src/main/java/com/oceanbase/odc/service/db/session/DefaultDBSessionManage.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

负责会话窗口的相关操作

server/odc-service/src/main/java/com/oceanbase/odc/service/db/DBFunctionService.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

从DBFunctionService开始都是对一些数据库对象的操作，

.../odc-service/src/main/java/com/oceanbase/odc/service/queryprofile/OBQueryProfileManager.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

负责性能分析，执行计划

...c-service/src/main/java/com/oceanbase/odc/service/rollbackplan/RollbackGeneratorFactory.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

负责事务回滚

...vice/src/main/java/com/oceanbase/odc/service/schedule/processor/AbstractDlmPreprocessor.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

主要负责数据归档任务创建，验证归档配置的有效性

...service/src/main/java/com/oceanbase/odc/service/session/OBQueryProfileExecutionListener.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

捕获traceId,负责OB数据库的性能分析

server/odc-service/src/main/java/com/oceanbase/odc/service/shadowtable/ShadowTableSyncTask.java
Copy link
Author

@littleniannian littleniannian Nov 17, 2025

Choose a reason for hiding this comment

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

用于批量同步影子表

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@littleniannian