Skip to content

[Bug] Fix bug of load error hub and schema change - #4486

Merged
morningman merged 2 commits into
apache:masterfrom
morningman:handle_mysql_client
Aug 31, 2020
Merged

[Bug] Fix bug of load error hub and schema change#4486
morningman merged 2 commits into
apache:masterfrom
morningman:handle_mysql_client

Conversation

@morningman

Copy link
Copy Markdown
Contributor

Proposed changes

  1. When WITH_MYSQL is off, load error hub does not suport MySQL load error hub,
    we should check its return value. (introduced by [License] Organize and modify the license of the code #4371 )

  2. misjudge the return value of change_row_block in schema_change.cpp (Introduced by Add OLAP_ERR_DATE_QUALITY_ERR error status to display schema change failure #4388 )

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

1. When WITH_MYSQL is off, load error hub does not suport MySQL load error hub,
we should check its return value.
2. misjudge the return value of `change_row_block` in schema_change.cpp
@morningmanmorningman added the kind/fix Categorizes issue or PR as related to a bug. label Aug 31, 2020
@morningmanmorningman self-assigned this Aug 31, 2020
Comment threadbe/src/olap/schema_change.cpp Outdated

@yangzhgyangzhg 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.

+1

@morningmanmorningman added the approved Indicates a PR has been approved by one committer. label Aug 31, 2020
@morningman
morningman merged commit a864db0 into apache:masterAug 31, 2020
ZhangYu0123 pushed a commit to ZhangYu0123/incubator-doris that referenced this pull request Sep 1, 2020
1. When WITH_MYSQL is off, load error hub does not suport MySQL load error hub,
we should check its return value.
2. misjudge the return value of `change_row_block` in schema_change.cpp
@yangzhgyangzhg mentioned this pull request Feb 9, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.kind/fixCategorizes issue or PR as related to a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@morningman@yangzhg@morningman-cmy