Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor convertToInt/ToInt64 and convertToUint/ToUint64. #464

Merged
merged 2 commits into from
Oct 29, 2015

Conversation

qiuyesuifeng
Copy link
Member

@siddontang
Copy link
Member

LGTM

}

func convertToUint(val interface{}, target *FieldType) (converted uint64, err error) {
func convertToUint(val interface{}, target *FieldType) (uint64, error) {
Copy link
Member

Choose a reason for hiding this comment

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

rename it to convertToUint64?

@shenli
Copy link
Member

shenli commented Oct 29, 2015

LGTM

qiuyesuifeng added a commit that referenced this pull request Oct 29, 2015
Refactor convertToInt/ToInt64 and convertToUint/ToUint64.
@qiuyesuifeng qiuyesuifeng merged commit 7ea5f16 into master Oct 29, 2015
@qiuyesuifeng qiuyesuifeng deleted the qiuyesuifeng/convert-cleanup branch October 29, 2015 07:23
nolouch pushed a commit to nolouch/tidb that referenced this pull request Jul 13, 2023
* serverless version 16

Signed-off-by: AmoebaProtozoa <[email protected]>

* fix typo in comments

Signed-off-by: AmoebaProtozoa <[email protected]>

* set special char to 0

Signed-off-by: AmoebaProtozoa <[email protected]>

---------

Signed-off-by: AmoebaProtozoa <[email protected]>
rleungx pushed a commit to rleungx/tidb that referenced this pull request Feb 26, 2024
* Adds a feature whitelist at planbuild stage (pingcap#304)

* init

Signed-off-by: David <[email protected]>

* format errors

Signed-off-by: David <[email protected]>

* restrict information schema

Signed-off-by: David <[email protected]>

* fmt

Signed-off-by: David <[email protected]>

* bazel

Signed-off-by: David <[email protected]>

* fmt

Signed-off-by: David <[email protected]>

* disable datadir

Signed-off-by: David <[email protected]>

* add ttl check for create and alter table

Signed-off-by: David <[email protected]>

* create table ttl

Signed-off-by: David <[email protected]>

* address comments

Signed-off-by: David <[email protected]>

* update error doc

Signed-off-by: David <[email protected]>

* add the back to the error message

Signed-off-by: David <[email protected]>

* unblock 'SHOW CONFIG' for lightning

Signed-off-by: David <[email protected]>

---------

Signed-off-by: David <[email protected]>
Co-authored-by: zzm <[email protected]>
Signed-off-by: AmoebaProtozoa <[email protected]>
# Conflicts:
#	ddl/BUILD.bazel
#	errno/errname.go
#	util/sem/sem.go

* add test flag that can disable feature control (pingcap#328)

Signed-off-by: David <[email protected]>
Co-authored-by: iosmanthus <[email protected]>

* Adds a feature whitelist at planbuild stage (pingcap#304)

* init

Signed-off-by: David <[email protected]>

* format errors

Signed-off-by: David <[email protected]>

* restrict information schema

Signed-off-by: David <[email protected]>

* fmt

Signed-off-by: David <[email protected]>

* bazel

Signed-off-by: David <[email protected]>

* fmt

Signed-off-by: David <[email protected]>

* disable datadir

Signed-off-by: David <[email protected]>

* add ttl check for create and alter table

Signed-off-by: David <[email protected]>

* create table ttl

Signed-off-by: David <[email protected]>

* address comments

Signed-off-by: David <[email protected]>

* update error doc

Signed-off-by: David <[email protected]>

* add the back to the error message

Signed-off-by: David <[email protected]>

* unblock 'SHOW CONFIG' for lightning

Signed-off-by: David <[email protected]>

---------

Signed-off-by: David <[email protected]>
Co-authored-by: zzm <[email protected]>
Signed-off-by: AmoebaProtozoa <[email protected]>
# Conflicts:
#	ddl/BUILD.bazel
#	errno/errname.go
#	util/sem/sem.go

* disable validate password (pingcap#465)

Signed-off-by: AmoebaProtozoa <[email protected]>

* turn on validate password and set special char count to 0 (pingcap#464)

* serverless version 16

Signed-off-by: AmoebaProtozoa <[email protected]>

* fix typo in comments

Signed-off-by: AmoebaProtozoa <[email protected]>

* set special char to 0

Signed-off-by: AmoebaProtozoa <[email protected]>

---------

Signed-off-by: AmoebaProtozoa <[email protected]>

* disabling password validation (pingcap#467)

* skip enabling password validation

Signed-off-by: AmoebaProtozoa <[email protected]>

* fix global variable

Signed-off-by: AmoebaProtozoa <[email protected]>

---------

Signed-off-by: AmoebaProtozoa <[email protected]>

* allow edit waittimeout (pingcap#468)

Signed-off-by: AmoebaProtozoa <[email protected]>

* allow edit  tidb-isolation-read-engines (pingcap#470)

Signed-off-by: AmoebaProtozoa <[email protected]>

* allow edit tidb_txn_mode (pingcap#471)

Signed-off-by: AmoebaProtozoa <[email protected]>

* allow edit tidb-cdc-write-source (pingcap#473)

Signed-off-by: AmoebaProtozoa <[email protected]>

* allow edit tidb_placement_mode (pingcap#474)

Signed-off-by: AmoebaProtozoa <[email protected]>

* please the checker

Signed-off-by: AmoebaProtozoa <[email protected]>

* sem enable now returns error on fail

Signed-off-by: AmoebaProtozoa <[email protected]>

* make check

Signed-off-by: AmoebaProtozoa <[email protected]>

---------

Signed-off-by: David <[email protected]>
Signed-off-by: AmoebaProtozoa <[email protected]>
Co-authored-by: iosmanthus <[email protected]>
Co-authored-by: zzm <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants