noth/surrealpatch
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

No need for a default TYPE on SQL DEFINE FIELD

Browse source 
Previously the default type for DEFINE FIELD statements was “any”, but this is unnecessary, as fields with type “any” are not type-checked anyway, so there is little point in defining the default as “any”. Therefore the type is left blank if no TYPE is specifically defined.
This commit is contained in:
Tobie Morgan Hitchcock 2016-11-26 13:35:58 +00:00
parent 90953735d9
commit 2b47e6e0a9
1 changed files with 0 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
sql/field.go
View file

@ -121,10 +121,6 @@ func (p *parser) parseDefineFieldStatement() (stmt *DefineFieldStatement, err er
} }
if stmt.Type == "" {
stmt.Type = "any"
}
if _, _, err = p.shouldBe(EOF, SEMICOLON); err != nil { if _, _, err = p.shouldBe(EOF, SEMICOLON); err != nil {
return nil, err return nil, err
} }