Report for github.com/factset/go-drill
A+ Excellent! Found 13 issues across 39 files
gofmt100%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
No problems detected. Good job!
go_vet100%
go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.
No problems detected. Good job!
gocyclo89%
Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.
-
-
go-drill/driver/connector.go
- Line 30: warning: cyclomatic complexity 21 of function parseConnectStr() is high (> 15) (gocyclo)
-
-
go-drill/internal/data/data_vector.go
- Line 275: warning: cyclomatic complexity 24 of function NewValueVec() is high (> 15) (gocyclo)
-
-
go-drill/internal/data/arrow.go
- Line 20: warning: cyclomatic complexity 21 of function ArrowTypeToReflect() is high (> 15) (gocyclo)
- Line 70: warning: cyclomatic complexity 19 of function TypeToArrowType() is high (> 15) (gocyclo)
-
-
go-drill/client.go
- Line 176: warning: cyclomatic complexity 24 of function (*Client).recvRoutine() is high (> 15) (gocyclo)
golint82%
Golint is a linter for Go source code.
-
-
go-drill/internal/log/log.go
- Line 11: warning: exported var Logger should have comment or be unexported (golint)
- Line 23: warning: exported function Printf should have comment or be unexported (golint)
- Line 27: warning: exported function Print should have comment or be unexported (golint)
-
-
go-drill/internal/data/data_vector.go
- Line 18: warning: exported type DataVector should have comment or be unexported (golint)
- Line 26: warning: exported type NullableDataVector should have comment or be unexported (golint)
- Line 32: warning: exported type BitVector should have comment or be unexported (golint)
- Line 38: warning: exported method BitVector.Type should have comment or be unexported (golint)
- Line 42: warning: exported method BitVector.TypeLen should have comment or be unexported (golint)
- Line 46: warning: exported method BitVector.Len should have comment or be unexported (golint)
- Line 50: warning: exported method BitVector.Get should have comment or be unexported (golint)
- Line 55: warning: exported method BitVector.Value should have comment or be unexported (golint)
- Line 59: warning: exported function NewBitVector should have comment or be unexported (golint)
- Line 67: warning: exported type NullableBitVector should have comment or be unexported (golint)
- Line 73: warning: exported method NullableBitVector.Get should have comment or be unexported (golint)
- Line 81: warning: exported method NullableBitVector.Value should have comment or be unexported (golint)
- Line 89: warning: exported function NewNullableBitVector should have comment or be unexported (golint)
- Line 99: warning: exported type VarbinaryVector should have comment or be unexported (golint)
- Line 107: warning: exported method VarbinaryVector.Type should have comment or be unexported (golint)
- Line 111: warning: exported method VarbinaryVector.TypeLen should have comment or be unexported (golint)
- Line 115: warning: exported method VarbinaryVector.Len should have comment or be unexported (golint)
- Line 119: warning: exported method VarbinaryVector.Get should have comment or be unexported (golint)
- Line 123: warning: exported method VarbinaryVector.Value should have comment or be unexported (golint)
- Line 127: warning: exported function NewVarbinaryVector should have comment or be unexported (golint)
- Line 161: warning: exported type VarcharVector should have comment or be unexported (golint)
- Line 165: warning: exported method VarcharVector.Type should have comment or be unexported (golint)
- Line 169: warning: exported method VarcharVector.Get should have comment or be unexported (golint)
- Line 174: warning: exported function NewVarcharVector should have comment or be unexported (golint)
- Line 178: warning: exported type NullableVarcharVector should have comment or be unexported (golint)
- Line 184: warning: exported method NullableVarcharVector.Get should have comment or be unexported (golint)
- Line 193: warning: exported method NullableVarcharVector.Value should have comment or be unexported (golint)
- Line 202: warning: exported function NewNullableVarcharVector should have comment or be unexported (golint)
- Line 212: warning: exported type DecimalVector should have comment or be unexported (golint)
- Line 220: warning: exported function NewDecimalVector should have comment or be unexported (golint)
- Line 229: warning: exported method DecimalVector.Get should have comment or be unexported (golint)
- Line 238: warning: exported method DecimalVector.Value should have comment or be unexported (golint)
- Line 242: warning: exported type NullableDecimalVector should have comment or be unexported (golint)
- Line 250: warning: exported method NullableDecimalVector.Get should have comment or be unexported (golint)
- Line 262: warning: exported method NullableDecimalVector.Value should have comment or be unexported (golint)
- Line 266: warning: exported function NewNullableDecimalVector should have comment or be unexported (golint)
- Line 275: warning: exported function NewValueVec should have comment or be unexported (golint)
-
-
go-drill/internal/data/date_time_vectors.go
- Line 13: warning: exported type TimestampVector should have comment or be unexported (golint)
- Line 17: warning: exported method TimestampVector.Type should have comment or be unexported (golint)
- Line 21: warning: exported function NewTimestampVector should have comment or be unexported (golint)
- Line 27: warning: exported method TimestampVector.Get should have comment or be unexported (golint)
- Line 32: warning: exported method TimestampVector.Value should have comment or be unexported (golint)
- Line 36: warning: exported type DateVector should have comment or be unexported (golint)
- Line 40: warning: exported method DateVector.Get should have comment or be unexported (golint)
- Line 44: warning: exported method DateVector.Value should have comment or be unexported (golint)
- Line 48: warning: exported function NewDateVector should have comment or be unexported (golint)
- Line 52: warning: exported type TimeVector should have comment or be unexported (golint)
- Line 56: warning: exported method TimeVector.Type should have comment or be unexported (golint)
- Line 60: warning: exported method TimeVector.Get should have comment or be unexported (golint)
- Line 66: warning: exported method TimeVector.Value should have comment or be unexported (golint)
- Line 70: warning: exported function NewTimeVector should have comment or be unexported (golint)
- Line 74: warning: exported type NullableTimestampVector should have comment or be unexported (golint)
- Line 78: warning: exported method NullableTimestampVector.Type should have comment or be unexported (golint)
- Line 82: warning: exported method NullableTimestampVector.Get should have comment or be unexported (golint)
- Line 92: warning: exported method NullableTimestampVector.Value should have comment or be unexported (golint)
- Line 101: warning: exported function NewNullableTimestampVector should have comment or be unexported (golint)
- Line 107: warning: exported type NullableDateVector should have comment or be unexported (golint)
- Line 111: warning: exported method NullableDateVector.Get should have comment or be unexported (golint)
- Line 120: warning: exported method NullableDateVector.Value should have comment or be unexported (golint)
- Line 129: warning: exported function NewNullableDateVector should have comment or be unexported (golint)
- Line 133: warning: exported type NullableTimeVector should have comment or be unexported (golint)
- Line 137: warning: exported method NullableTimeVector.Type should have comment or be unexported (golint)
- Line 141: warning: exported method NullableTimeVector.Get should have comment or be unexported (golint)
- Line 152: warning: exported method NullableTimeVector.Value should have comment or be unexported (golint)
- Line 161: warning: exported function NewNullableTimeVector should have comment or be unexported (golint)
- Line 336: warning: exported function NewIntervalYearVector should have comment or be unexported (golint)
- Line 343: warning: exported function NewNullableIntervalYearVector should have comment or be unexported (golint)
- Line 350: warning: exported function NewIntervalDayVector should have comment or be unexported (golint)
- Line 357: warning: exported function NewNullableIntervalDayVector should have comment or be unexported (golint)
- Line 364: warning: exported function NewIntervalVector should have comment or be unexported (golint)
- Line 371: warning: exported function NewNullableIntervalVector should have comment or be unexported (golint)
-
-
go-drill/internal/data/decimal_utils.go
- Line 9: warning: exported type DecimalTraits should have comment or be unexported (golint)
- Line 17: warning: exported var Decimal28DenseTraits should have comment or be unexported (golint)
-
-
go-drill/client.go
- Line 24: warning: exported type QueryType should have comment or be unexported (golint)
- Line 27: warning: exported const TypeSQL should have comment (or a comment on this block) or be unexported (golint)
-
-
go-drill/result_handle.go
- Line 20: warning: exported type DataVector should have comment or be unexported (golint)
- Line 21: warning: exported type NullableDataVector should have comment or be unexported (golint)
-
-
go-drill/utils.go
- Line 140: warning: exported type ColumnMeta should have comment or be unexported (golint)
ineffassign97%
IneffAssign detects ineffectual assignments in Go code.
-
-
go-drill/client_test.go
- Line 92: warning: ineffectual assignment to err (ineffassign)
misspell89%
Misspell Finds commonly misspelled English words
-
-
go-drill/kerberos_smoke_test.go
- Line 48: warning: "carefull" is a misspelling of "careful" (misspell)
-
-
go-drill/result_handle.go
- Line 325: warning: "recieved" is a misspelling of "received" (misspell)
- Line 338: warning: "recieved" is a misspelling of "received" (misspell)
-
-
go-drill/result_handle_test.go
- Line 118: warning: "carefull" is a misspelling of "careful" (misspell)
- Line 132: warning: "carefull" is a misspelling of "careful" (misspell)
- Line 334: warning: "carefull" is a misspelling of "careful" (misspell)
- Line 348: warning: "carefull" is a misspelling of "careful" (misspell)
-
-
go-drill/smoke_test.go
- Line 48: warning: "carefull" is a misspelling of "careful" (misspell)