Skip to content

types: add overflow truncate when convert str to float#5130

Merged
winoros merged 3 commits intopingcap:masterfrom
mccxj:str2float
Nov 22, 2017
Merged

types: add overflow truncate when convert str to float#5130
winoros merged 3 commits intopingcap:masterfrom
mccxj:str2float

Conversation

@mccxj
Copy link
Contributor

@mccxj mccxj commented Nov 16, 2017

fix #5111 . @winoros PTAL.

@sre-bot
Copy link
Contributor

sre-bot commented Nov 16, 2017

Hi contributor, thanks for your PR.

This patch needs to be approved by someone of admins. They should reply with "/ok-to-test" to accept this PR for running test automatically.

@shenli
Copy link
Member

shenli commented Nov 16, 2017

@mccxj Thanks!

@shenli shenli added WIP contribution This PR is from a community contributor. labels Nov 16, 2017
@mccxj
Copy link
Contributor Author

mccxj commented Nov 18, 2017

@winoros PTAL.

Copy link
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

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

LGTM

@zz-jason zz-jason added status/LGT1 Indicates that a PR has LGTM 1. and removed WIP labels Nov 22, 2017
Copy link
Member

@winoros winoros left a comment

Choose a reason for hiding this comment

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

LGTM

@winoros
Copy link
Member

winoros commented Nov 22, 2017

/run-all-tests

@winoros
Copy link
Member

winoros commented Nov 22, 2017

/run-integration-ddl-test

@winoros winoros added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Nov 22, 2017
@winoros winoros merged commit 9921f41 into pingcap:master Nov 22, 2017
@mccxj mccxj deleted the str2float branch November 22, 2017 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution This PR is from a community contributor. status/LGT2 Indicates that a PR has LGTM 2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error executing query with varchar column

5 participants