Skip to content

Conversation

@leaderwolfpipi
Copy link
Collaborator

No description provided.

@SunnyHaze
Copy link
Collaborator

  • move exceptions and imports inside class. type annotation also removed for lazzyload click_house

Copy link
Collaborator

@SunnyHaze SunnyHaze left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@SunnyHaze SunnyHaze changed the title Main 0923 [KBC, db_pool] fix registry bug in KBC pipeline; add myscale db_pool to support connect pool for myscale Sep 25, 2025
SunnyHaze added a commit that referenced this pull request Sep 25, 2025
…to support connect pool for myscale #271 (#275)

* myscale换成连接池形式

* 补充部分算子的注册逻辑,添加myscale连接池并调整目录结构

* [db_pool] add lazy load for clickhouse driver

* [db_pool] rm type comments for lazy load

* 修复作用域错误导致的ClickHouseError使用错误

---------

Co-authored-by: Sunnyhaze <mxch1122@126.com>
MOLYHECI referenced this pull request in MOLYHECI/DataFlow-1022 Oct 16, 2025
…to support connect pool for myscale #271 (#275)

* myscale换成连接池形式

* 补充部分算子的注册逻辑,添加myscale连接池并调整目录结构

* [db_pool] add lazy load for clickhouse driver

* [db_pool] rm type comments for lazy load

* 修复作用域错误导致的ClickHouseError使用错误

---------

Co-authored-by: Sunnyhaze <mxch1122@126.com>
wongzhenhao pushed a commit to wongzhenhao/DataFlow that referenced this pull request Oct 17, 2025
…to support connect pool for myscale OpenDCAI#271 (OpenDCAI#275)

* myscale换成连接池形式

* 补充部分算子的注册逻辑,添加myscale连接池并调整目录结构

* [db_pool] add lazy load for clickhouse driver

* [db_pool] rm type comments for lazy load

* 修复作用域错误导致的ClickHouseError使用错误

---------

Co-authored-by: Sunnyhaze <mxch1122@126.com>
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.

2 participants