Skip to content

fix(osc): exists horizontal overstep access data permission when swap table manual#1405

Merged
yhilmare merged 6 commits into
dev/4.2.3from
yaobin_osc_swap_table_fix_horizontal_permisson
Jan 17, 2024
Merged

fix(osc): exists horizontal overstep access data permission when swap table manual#1405
yhilmare merged 6 commits into
dev/4.2.3from
yaobin_osc_swap_table_fix_horizontal_permisson

Conversation

@krihy
Copy link
Copy Markdown
Contributor

@krihy krihy commented Jan 16, 2024

What type of PR is this?

type-bug
module-onlineSchemaChange

What this PR does / why we need it:

exists horizontal overstep access data permission when swap table manual

Which issue(s) this PR fixes:

add permission valid, only flow instance creator can operate swap table name.

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 16, 2024
@krihy krihy added this to the ODC 4.2.3-bp milestone Jan 16, 2024
@krihy krihy self-assigned this Jan 16, 2024
@krihy krihy requested a review from yhilmare January 16, 2024 11:33
smallsheeeep
smallsheeeep previously approved these changes Jan 17, 2024
Copy link
Copy Markdown
Contributor

@smallsheeeep smallsheeeep left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@yhilmare yhilmare 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 d54225b into dev/4.2.3 Jan 17, 2024
@yhilmare yhilmare deleted the yaobin_osc_swap_table_fix_horizontal_permisson branch January 17, 2024 06:45
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 type-bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants