Skip to content

[fix](regression) add "enable_unique_key_merge_on_write" = "false" for cases that want to use MOR unique table - #28090

Closed
bobhan1 wants to merge 2 commits into
apache:masterfrom
bobhan1:fix-mor
Closed

[fix](regression) add "enable_unique_key_merge_on_write" = "false" for cases that want to use MOR unique table#28090
bobhan1 wants to merge 2 commits into
apache:masterfrom
bobhan1:fix-mor

Conversation

@bobhan1

Copy link
Copy Markdown
Contributor

Proposed changes

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@bobhan1bobhan1 changed the title [fix](regression) add "enable_unique_key_merge_on_write" = "false" for cases that want to uee MOR unique table[fix](regression) add "enable_unique_key_merge_on_write" = "false" for cases that want to use MOR unique tableDec 6, 2023
@bobhan1

Copy link
Copy Markdown
ContributorAuthor

run buildall

eldenmoon
eldenmoon previously approved these changes Dec 6, 2023

@eldenmooneldenmoon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Dec 6, 2023
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@bobhan1

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actionsgithub-actionsBot removed the approved Indicates a PR has been approved by one committer. label Dec 7, 2023
@bobhan1

Copy link
Copy Markdown
ContributorAuthor

run p0

@bobhan1

Copy link
Copy Markdown
ContributorAuthor

#28105

@bobhan1bobhan1 closed this Dec 7, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@bobhan1@eldenmoon