Skip to content

fix(osc): osc task don't show manual swap table name when full migrate is completed#1357

Merged
yhilmare merged 4 commits into
dev/4.2.3from
yaobin_fix_osc_manual_swap_table_name
Jan 11, 2024
Merged

fix(osc): osc task don't show manual swap table name when full migrate is completed#1357
yhilmare merged 4 commits into
dev/4.2.3from
yaobin_fix_osc_manual_swap_table_name

Conversation

@krihy
Copy link
Copy Markdown
Contributor

@krihy krihy commented Jan 8, 2024

What type of PR is this?

type-bug
module-online schema change

What this PR does / why we need it:

fix osc percentage continue execute when isManualSwapTableEnabled is true, we should update it to task result json, the swap table button show or not based on isManualSwapTableEnabled

Which issue(s) this PR fixes:

Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:


@krihy krihy added type-bug Something isn't working module-Online schema change Online schema change labels Jan 8, 2024
@krihy krihy added this to the ODC 4.2.3-bp milestone Jan 8, 2024
@krihy krihy self-assigned this Jan 8, 2024
@krihy krihy changed the base branch from dev/4.2.x to dev/4.2.3 January 8, 2024 11:06
@krihy krihy requested a review from MarkPotato777 January 8, 2024 11:08
@krihy krihy requested a review from guowl3 January 11, 2024 06:11
Copy link
Copy Markdown
Collaborator

@guowl3 guowl3 left a comment

Choose a reason for hiding this comment

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

LGTM

@yhilmare yhilmare merged commit 7d1f7f0 into dev/4.2.3 Jan 11, 2024
@yhilmare yhilmare deleted the yaobin_fix_osc_manual_swap_table_name branch January 11, 2024 10:59
krihy added a commit that referenced this pull request Jan 12, 2024
…e is completed (#1357)

* fix manual swap table

* record manualSwapTableTasksChanged

* fix swap table list

* format code
krihy added a commit that referenced this pull request Jan 12, 2024
…e is completed (#1357)

* fix manual swap table

* record manualSwapTableTasksChanged

* fix swap table list

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

Labels

module-Online schema change Online schema change priority-high type-bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[Bug]: osc full migrate is completed and not show swap table name button when swap table is manual model

3 participants