Skip to content

[To rc/1.3.3] AINode: Refined architecture for enhanced modularity & Added basic IT#13470

Merged
SteveYurongSu merged 1 commit intorc/1.3.3from
refactorTo1.3.3
Sep 11, 2024
Merged

[To rc/1.3.3] AINode: Refined architecture for enhanced modularity & Added basic IT#13470
SteveYurongSu merged 1 commit intorc/1.3.3from
refactorTo1.3.3

Conversation

@ycycse
Copy link
Copy Markdown
Member

@ycycse ycycse commented Sep 11, 2024

cherrypick

…#13423)

This PR refactors the AINode codebase by breaking down the existing classes into separate modules, enhancing code readability and maintainability. Additionally, I have identified and removed code smells, ensuring cleanliness and consistency throughout the code. These improvements will facilitate future feature expansions and bug fixes.

------

* basic refactor the logic of manager

* refactor ainode (has bug needed to be fixed)

* use logger that is uniformly initialized.

* support singleton for ClientManager

* Refine the locking mechanism for multi-threading

* Add IT of model for AINode

* ignore AINode-example in header check

* refine dropModelTest which may exist error

* optimized cluster number to reduce execution time

* class-level method needs to be static

* try to fix CI

* fix the problem where CREATE MODEL may be slow.

* reduce cluster number

(cherry picked from commit cfc4eee)
@SteveYurongSu SteveYurongSu self-assigned this Sep 11, 2024
@SteveYurongSu SteveYurongSu changed the title [TO 1.3.3] AINode: Refined architecture for enhanced modularity & Added basic IT [To rc/1.3.3] AINode: Refined architecture for enhanced modularity & Added basic IT Sep 11, 2024
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.45%. Comparing base (8b159b1) to head (bed1f27).
Report is 4 commits behind head on rc/1.3.3.

Additional details and impacted files
@@              Coverage Diff               @@
##             rc/1.3.3   #13470      +/-   ##
==============================================
- Coverage       41.46%   41.45%   -0.01%     
  Complexity         71       71              
==============================================
  Files            3445     3445              
  Lines          219730   219730              
  Branches        26135    26135              
==============================================
- Hits            91104    91089      -15     
- Misses         128626   128641      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SteveYurongSu SteveYurongSu merged commit 8662432 into rc/1.3.3 Sep 11, 2024
@SteveYurongSu SteveYurongSu deleted the refactorTo1.3.3 branch September 11, 2024 03:07
Caideyipi pushed a commit to Caideyipi/iotdb that referenced this pull request Mar 25, 2026
…apache#13423) (apache#13470)

This PR refactors the AINode codebase by breaking down the existing classes into separate modules, enhancing code readability and maintainability. Additionally, I have identified and removed code smells, ensuring cleanliness and consistency throughout the code. These improvements will facilitate future feature expansions and bug fixes.

------

* basic refactor the logic of manager

* refactor ainode (has bug needed to be fixed)

* use logger that is uniformly initialized.

* support singleton for ClientManager

* Refine the locking mechanism for multi-threading

* Add IT of model for AINode

* ignore AINode-example in header check

* refine dropModelTest which may exist error

* optimized cluster number to reduce execution time

* class-level method needs to be static

* try to fix CI

* fix the problem where CREATE MODEL may be slow.

* reduce cluster number

(cherry picked from commit cfc4eee)
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