Skip to content

[Bug] Fix bug of select @@sql_mode - #4484

Merged
morningman merged 1 commit into
apache:masterfrom
morningman:sql_mode_type
Sep 1, 2020
Merged

[Bug] Fix bug of select @@sql_mode#4484
morningman merged 1 commit into
apache:masterfrom
morningman:sql_mode_type

Conversation

@morningman

@morningmanmorningman commented Aug 29, 2020•
edited
Loading

Copy link
Copy Markdown
Contributor

Proposed changes

Fix bug that select @@sql_mode throw error: Invalid number format.

Types of changes

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

Checklist

  • [] I have create an issue on (Fixconnect doris use mysql-driver error #4483), and have described the bug/feature there in detail
  • [] Compiling and unit tests pass locally with my changes
  • [] I have added tests that prove my fix is effective or that my feature works
  • [] If this change need a document change, I have updated the document
  • [] Any dependent changes have been merged

Further comments

Not sure this is a good solution.

@morningmanmorningman added kind/fix Categorizes issue or PR as related to a bug. area/mysql-compatibility Issues or PRs related to the mysql compatibility labels Aug 29, 2020
yangzhg
yangzhg approved these changes Aug 31, 2020
View reviewed changes

@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 d495661 into apache:masterSep 1, 2020
ZhangYu0123 pushed a commit to ZhangYu0123/incubator-doris that referenced this pull request Sep 1, 2020
@morningman@ZhangYu0123
[Bug] Fix bug of select @@sql_mode (apache#4484)
362ac45
Fix bug that `select @@sql_mode` throw error: Invalid number format.
@yangzhgyangzhg mentioned this pull request Feb 9, 2021
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yangzhgyangzhgyangzhg approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.area/mysql-compatibilityIssues or PRs related to the mysql compatibilitykind/fixCategorizes issue or PR as related to a bug.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

connect doris use mysql-driver error

2 participants

@morningman@yangzhg