MyFavMirrors/minio
1
0
Fork 0
mirror of https://github.com/minio/minio.git synced 2025-04-04 11:50:36 -04:00
Code Issues Packages Projects Releases Wiki Activity

select: Allow lower case 'is' (#14405)

Browse Source 
Ref: #14358
This commit is contained in:
Klaus Post 2022-02-24 09:10:48 -08:00 committed by GitHub
parent c08540c7b7
commit 2cea944cdb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
internal/s3select/select_test.go
View File

@ -474,6 +474,13 @@ func TestJSONQueries(t *testing.T) {
wantResult: `{"Id":1,"Name":"Anshu","Address":"Templestowe","Car":"Jeep"}`, wantResult: `{"Id":1,"Name":"Anshu","Address":"Templestowe","Car":"Jeep"}`,
withJSON: `{ "person": [ { "Id": 1, "Name": "Anshu", "Address": "Templestowe", "Car": "Jeep" }, { "Id": 2, "Name": "Ben Mostafa", "Address": "Las Vegas", "Car": "Mustang" }, { "Id": 3, "Name": "Rohan Wood", "Address": "Wooddon", "Car": "VW" } ] }`, withJSON: `{ "person": [ { "Id": 1, "Name": "Anshu", "Address": "Templestowe", "Car": "Jeep" }, { "Id": 2, "Name": "Ben Mostafa", "Address": "Las Vegas", "Car": "Mustang" }, { "Id": 3, "Name": "Rohan Wood", "Address": "Wooddon", "Car": "VW" } ] }`,
}, },
{
name: "lower-case-is",
query: `select * from s3object[*] as s where s.request.header['User-Agent'] is not null`,
wantResult: `{"request":{"uri":"/1","header":{"User-Agent":"test"}}}`,
withJSON: `{"request":{"uri":"/1","header":{"User-Agent":"test"}}}
{"request":{"uri":"/2","header":{}}}`,
},
} }
defRequest := `<?xml version="1.0" encoding="UTF-8"?> defRequest := `<?xml version="1.0" encoding="UTF-8"?>

3
internal/s3select/sql/evaluate.go
View File

@ -22,6 +22,7 @@ import (
"errors" "errors"
"fmt" "fmt"
"math" "math"
"strings"
"github.com/bcicen/jstream" "github.com/bcicen/jstream"
"github.com/minio/simdjson-go" "github.com/minio/simdjson-go"
@ -134,7 +135,7 @@ func (e *ConditionOperand) evalNode(r Record, tableAlias string) (*Value, error)
return nil, cmpRErr return nil, cmpRErr
} }
b, err := opVal.compareOp(e.ConditionRHS.Compare.Operator, cmpRight) b, err := opVal.compareOp(strings.ToUpper(e.ConditionRHS.Compare.Operator), cmpRight)
return FromBool(b), err return FromBool(b), err
case e.ConditionRHS.Between != nil: case e.ConditionRHS.Between != nil: