Skip to content

Conversation

@tezc
Copy link
Owner

@tezc tezc commented Aug 18, 2025

@ShooterIT I was accumulating some changes on my trim branch, it is getting bigger. So, I believe we can merge some of them separately:

Changes:

  1. Added [ID <task_id] parameter to CLUSTER MIGRATION STATUS. If specified, only the given task is returned. This is just for convenience if task list is crowded.
  2. Moved some slotRangeArray functions together and added a few more for convenience. We better have their declaration in cluster.h so that plugin may use if needed.
  3. Added a debug parameter to disable trim. In a later PR, we can have more options to switch between active and background trim.
  4. Added asmGenInfoString() to be able to generate info string for both plugin and legacy implementations.
  5. Added some helper functions to tests.

@tezc tezc requested a review from ShooterIT August 18, 2025 11:20
Copy link
Collaborator

@ShooterIT ShooterIT left a comment

Choose a reason for hiding this comment

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

excellent work, i like your code

@tezc tezc merged commit 3c932e3 into cluster-asm Aug 19, 2025
26 of 30 checks passed
@tezc tezc deleted the cluster-asm-improv branch August 19, 2025 12:01
tezc added a commit that referenced this pull request Sep 10, 2025
Some improvements and utility functions
tezc added a commit that referenced this pull request Sep 16, 2025
Some improvements and utility functions
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.

3 participants