Skip to content
This repository has been archived by the owner on Jan 28, 2021. It is now read-only.

Check correct usage of Null value #542

Closed
ajnavarro opened this issue Nov 8, 2018 · 1 comment · Fixed by #555
Closed

Check correct usage of Null value #542

ajnavarro opened this issue Nov 8, 2018 · 1 comment · Fixed by #555
Assignees
@kuba--
Labels
bug Something isn't working

Comments

@ajnavarro
Copy link
Contributor

ajnavarro commented Nov 8, 2018 •
edited
Loading

Null type here: https://github.com/src-d/go-mysql-server/blob/master/sql/type.go#L152 shouldn't be a type in most of the cases. Null values always should have a type assigned. Null type should be used only when no other type is inferred, per example: SELECT NULL query.
@ajnavarro ajnavarro added the bug Something isn't working label Nov 8, 2018
@ajnavarro ajnavarro changed the title Remove Null type Check correct usage of Null value Nov 8, 2018
@kuba-- kuba-- self-assigned this Nov 20, 2018
@kuba--
Copy link
Contributor

kuba-- commented Nov 20, 2018

Can be related: #546

@kuba-- kuba-- mentioned this issue Nov 20, 2018
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kuba-- kuba--

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement ifnull and nullif functions.
2 participants
@ajnavarro @kuba--