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!

gocyclo78%

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.

• parser/sqlparser/analyzer.go
  • Line 224: warning: cyclomatic complexity 19 of function ExtractSetValues() is high (> 15) (gocyclo)
  • Line 57: warning: cyclomatic complexity 17 of function Preview() is high (> 15) (gocyclo)

• parser/sqlparser/dependency/sqltypes/value.go
  • Line 71: warning: cyclomatic complexity 19 of function BuildValue() is high (> 15) (gocyclo)

• outputs/stream/internally_consistent_output_stream_wrapper.go
  • Line 14: warning: cyclomatic complexity 17 of function (*InternallyConsistentOutputStreamWrapper).Run() is high (> 15) (gocyclo)

• parser/parser.go
  • Line 542: warning: cyclomatic complexity 51 of function ParseExpression() is high (> 15) (gocyclo)
  • Line 55: warning: cyclomatic complexity 44 of function ParseSelect() is high (> 15) (gocyclo)

• physical/nodes.go
  • Line 246: warning: cyclomatic complexity 41 of function (*Node).Materialize() is high (> 15) (gocyclo)

• logical/utils.go
  • Line 259: warning: cyclomatic complexity 32 of function EqualExpressions() is high (> 15) (gocyclo)

• outputs/batch/live_output.go
  • Line 75: warning: cyclomatic complexity 17 of function (*OutputPrinter).Run() is high (> 15) (gocyclo)

• parser/sqlparser/ast.go
  • Line 1172: warning: cyclomatic complexity 45 of function (*ColumnType).SQLType() is high (> 15) (gocyclo)
  • Line 795: warning: cyclomatic complexity 20 of function (*DDL).Format() is high (> 15) (gocyclo)
  • Line 1084: warning: cyclomatic complexity 20 of function (*ColumnType).Format() is high (> 15) (gocyclo)

• datasources/json/execution.go
  • Line 50: warning: cyclomatic complexity 32 of function getOctoSQLValue() is high (> 15) (gocyclo)

• physical/explain.go
  • Line 11: warning: cyclomatic complexity 25 of function ExplainNode() is high (> 15) (gocyclo)
  • Line 148: warning: cyclomatic complexity 17 of function ExplainExpr() is high (> 15) (gocyclo)

• datasources/csv/impl.go
  • Line 17: warning: cyclomatic complexity 19 of function Creator() is high (> 15) (gocyclo)

• parser/sqlparser/dependency/sqltypes/type.go
  • Line 148: warning: cyclomatic complexity 19 of function modifyType() is high (> 15) (gocyclo)

• datasources/csv/execution.go
  • Line 22: warning: cyclomatic complexity 19 of function (*DatasourceExecuting).Run() is high (> 15) (gocyclo)

• optimizer/push_filter_into_stream_join_key.go
  • Line 8: warning: cyclomatic complexity 17 of function PushDownFilterPredicatesIntoStreamJoinKey() is high (> 15) (gocyclo)

• physical/expression.go
  • Line 115: warning: cyclomatic complexity 32 of function (*Expression).Materialize() is high (> 15) (gocyclo)

• logical/map.go
  • Line 24: warning: cyclomatic complexity 22 of function (*Map).Typecheck() is high (> 15) (gocyclo)

• logical/group_by.go
  • Line 93: warning: cyclomatic complexity 21 of function (*GroupBy).Typecheck() is high (> 15) (gocyclo)

• logical/function.go
  • Line 24: warning: cyclomatic complexity 24 of function (*FunctionExpression).Typecheck() is high (> 15) (gocyclo)

• execution/nodes/stream_join.go
  • Line 40: warning: cyclomatic complexity 38 of function (*StreamJoin).Run() is high (> 15) (gocyclo)
  • Line 287: warning: cyclomatic complexity 19 of function (*StreamJoin).receiveRecord() is high (> 15) (gocyclo)

• octosql/values.go
  • Line 94: warning: cyclomatic complexity 43 of function (Value).Compare() is high (> 15) (gocyclo)
  • Line 297: warning: cyclomatic complexity 18 of function (Value).append() is high (> 15) (gocyclo)

• octosql/types.go
  • Line 66: warning: cyclomatic complexity 29 of function (Type).Is() is high (> 15) (gocyclo)
  • Line 208: warning: cyclomatic complexity 18 of function TypeSum() is high (> 15) (gocyclo)
  • Line 149: warning: cyclomatic complexity 17 of function (Type).String() is high (> 15) (gocyclo)

• plugins/manager/manager.go
  • Line 113: warning: cyclomatic complexity 28 of function (*PluginManager).Install() is high (> 15) (gocyclo)

• execution/nodes/group_by.go
  • Line 64: warning: cyclomatic complexity 19 of function (*GroupBy).Run() is high (> 15) (gocyclo)

• functions/functions.go
  • Line 18: warning: cyclomatic complexity 70 of function FunctionMap() is high (> 15) (gocyclo)

• parser/sqlparser/token.go
  • Line 487: warning: cyclomatic complexity 57 of function (*Tokenizer).Scan() is high (> 15) (gocyclo)
  • Line 842: warning: cyclomatic complexity 16 of function (*Tokenizer).scanString() is high (> 15) (gocyclo)

• logical/tvf.go
  • Line 122: warning: cyclomatic complexity 43 of function (*TableValuedFunction).Typecheck() is high (> 15) (gocyclo)

• physical/transform.go
  • Line 9: warning: cyclomatic complexity 24 of function (*Transformers).TransformNode() is high (> 15) (gocyclo)
  • Line 200: warning: cyclomatic complexity 17 of function (*Transformers).TransformExpr() is high (> 15) (gocyclo)

golint23%

Golint is a linter for Go source code.

• aggregates/max.go
  • Line 14: warning: exported var MaxOverloads should have comment or be unexported (golint)
  • Line 37: warning: exported type Max should have comment or be unexported (golint)
  • Line 41: warning: exported function NewMaxPrototype should have comment or be unexported (golint)
  • Line 63: warning: exported method Max.Add should have comment or be unexported (golint)
  • Line 88: warning: exported method Max.Trigger should have comment or be unexported (golint)

• datasources/docs/aggregate_signatures.go
  • Line 10: warning: should not use dot imports (golint)

• datasources/plugins/installed_plugins.go
  • Line 8: warning: should not use dot imports (golint)

• optimizer/filter_merge.go
  • Line 5: warning: should not use dot imports (golint)
  • Line 8: warning: exported function MergeFilters should have comment or be unexported (golint)
  • Line 40: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• logical/map.go
  • Line 12: warning: exported type Map should have comment or be unexported (golint)
  • Line 20: warning: exported function NewMap should have comment or be unexported (golint)
  • Line 24: warning: exported method Map.Typecheck should have comment or be unexported (golint)

• telemetry/query.go
  • Line 8: warning: exported type QueryTelemetryData should have comment or be unexported (golint)
  • Line 17: warning: exported function GetQueryTelemetryData should have comment or be unexported (golint)

• datasources/plugins/database.go
  • Line 7: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 16: warning: exported function Creator should have comment or be unexported (golint)
  • Line 23: warning: exported type Database should have comment or be unexported (golint)
  • Line 28: warning: exported method Database.ListTables should have comment or be unexported (golint)
  • Line 38: warning: exported method Database.GetTable should have comment or be unexported (golint)

• octosql/values.go
  • Line 9: warning: exported var ZeroValue should have comment or be unexported (golint)
  • Line 25: warning: exported function NewNull should have comment or be unexported (golint)
  • Line 31: warning: exported function NewInt should have comment or be unexported (golint)
  • Line 38: warning: exported function NewFloat should have comment or be unexported (golint)
  • Line 45: warning: exported function NewBoolean should have comment or be unexported (golint)
  • Line 52: warning: exported function NewString should have comment or be unexported (golint)
  • Line 59: warning: exported function NewTime should have comment or be unexported (golint)
  • Line 66: warning: exported function NewDuration should have comment or be unexported (golint)
  • Line 73: warning: exported function NewList should have comment or be unexported (golint)
  • Line 80: warning: exported function NewStruct should have comment or be unexported (golint)
  • Line 87: warning: exported function NewTuple should have comment or be unexported (golint)
  • Line 94: warning: exported method Value.Compare should have comment or be unexported (golint)
  • Line 100: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 239: warning: exported method Value.Equal should have comment or be unexported (golint)
  • Line 246: warning: exported method Value.Type should have comment or be unexported (golint)
  • Line 360: warning: exported method Value.ToRawGoValue should have comment or be unexported (golint)

• parser/parser.go
  • Line 49: warning: exported type OutputOptions should have comment or be unexported (golint)
  • Line 55: warning: exported function ParseSelect should have comment or be unexported (golint)
  • Line 263: warning: exported function ParseWith should have comment or be unexported (golint)
  • Line 283: warning: exported function ParseNode should have comment or be unexported (golint)
  • Line 303: warning: exported function ParseTableExpression should have comment or be unexported (golint)
  • Line 318: warning: exported function ParseAliasedTableExpression should have comment or be unexported (golint)
  • Line 353: warning: exported function ParseJoinTableExpression should have comment or be unexported (golint)
  • Line 407: warning: exported function ParseTableValuedFunction should have comment or be unexported (golint)
  • Line 424: warning: exported function ParseTableValuedFunctionArgument should have comment or be unexported (golint)
  • Line 452: warning: exported var ErrNotAggregate should have comment or be unexported (golint)
  • Line 454: warning: exported function ParseAggregate should have comment or be unexported (golint)
  • Line 491: warning: exported function ParseTrigger should have comment or be unexported (golint)
  • Line 524: warning: exported function ParseAliasedExpression should have comment or be unexported (golint)
  • Line 533: warning: exported function ParseFunctionArgument should have comment or be unexported (golint)
  • Line 542: warning: exported function ParseExpression should have comment or be unexported (golint)
  • Line 738: warning: exported function ParseType should have comment or be unexported (golint)
  • Line 806: warning: exported function ParseInfixOperator should have comment or be unexported (golint)
  • Line 824: warning: exported function ParsePrefixOperator should have comment or be unexported (golint)
  • Line 832: warning: exported function ParseInfixComparison should have comment or be unexported (golint)

• execution/record_event_time_buffer.go
  • Line 10: warning: exported type RecordEventTimeBuffer should have comment or be unexported (golint)
  • Line 14: warning: exported function NewRecordEventTimeBuffer should have comment or be unexported (golint)
  • Line 34: warning: exported method RecordEventTimeBuffer.AddRecord should have comment or be unexported (golint)
  • Line 52: warning: exported method RecordEventTimeBuffer.Emit should have comment or be unexported (golint)

• aggregates/count.go
  • Line 9: warning: exported var CountOverloads should have comment or be unexported (golint)
  • Line 17: warning: exported type Count should have comment or be unexported (golint)
  • Line 21: warning: exported function NewCountPrototype should have comment or be unexported (golint)
  • Line 29: warning: exported method Count.Add should have comment or be unexported (golint)
  • Line 38: warning: exported method Count.Trigger should have comment or be unexported (golint)

• datasources/docs/database.go
  • Line 7: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 13: warning: exported function Creator should have comment or be unexported (golint)
  • Line 17: warning: exported type Database should have comment or be unexported (golint)
  • Line 20: warning: exported method Database.ListTables should have comment or be unexported (golint)
  • Line 26: warning: exported method Database.GetTable should have comment or be unexported (golint)

• datasources/plugins/installed_versions.go
  • Line 8: warning: should not use dot imports (golint)

• execution/nodes/stream_join.go
  • Line 9: warning: should not use dot imports (golint)
  • Line 13: warning: exported type StreamJoin should have comment or be unexported (golint)
  • Line 18: warning: exported function NewStreamJoin should have comment or be unexported (golint)
  • Line 40: warning: exported method StreamJoin.Run should have comment or be unexported (golint)
  • Line 359: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• optimizer/filter_requalifier_pushdown.go
  • Line 4: warning: should not use dot imports (golint)
  • Line 7: warning: exported function PushDownFilterUnderRequalifier should have comment or be unexported (golint)
  • Line 68: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• outputs/batch/live_output.go
  • Line 12: warning: should not use dot imports (golint)
  • Line 17: warning: exported type Format should have comment or be unexported (golint)
  • Line 23: warning: exported type OutputPrinter should have comment or be unexported (golint)
  • Line 34: warning: exported function NewOutputPrinter should have comment or be unexported (golint)
  • Line 75: warning: exported method OutputPrinter.Run should have comment or be unexported (golint)

• logical/function.go
  • Line 12: warning: exported type FunctionExpression should have comment or be unexported (golint)
  • Line 17: warning: exported function NewFunctionExpression should have comment or be unexported (golint)
  • Line 24: warning: exported method FunctionExpression.Typecheck should have comment or be unexported (golint)

• execution/nodes/event_time_buffer.go
  • Line 6: warning: should not use dot imports (golint)
  • Line 9: warning: exported type EventTimeBuffer should have comment or be unexported (golint)
  • Line 13: warning: exported function NewEventTimeBuffer should have comment or be unexported (golint)
  • Line 17: warning: exported method EventTimeBuffer.Run should have comment or be unexported (golint)

• execution/nodes/group_by.go
  • Line 9: warning: should not use dot imports (golint)
  • Line 13: warning: exported type GroupBy should have comment or be unexported (golint)
  • Line 22: warning: exported function NewGroupBy should have comment or be unexported (golint)
  • Line 42: warning: exported type Aggregate should have comment or be unexported (golint)
  • Line 64: warning: exported method GroupBy.Run should have comment or be unexported (golint)

• execution/nodes/order_by.go
  • Line 9: warning: should not use dot imports (golint)
  • Line 15: warning: exported type OrderBy should have comment or be unexported (golint)
  • Line 21: warning: exported function NewBatchOrderBy should have comment or be unexported (golint)
  • Line 58: warning: exported method OrderBy.Run should have comment or be unexported (golint)

• cmd/root.go
  • Line 42: warning: exported var VERSION should have comment or be unexported (golint)
  • Line 362: warning: exported function Execute should have comment or be unexported (golint)

• logical/requalifier.go
  • Line 12: warning: exported type Requalifier should have comment or be unexported (golint)
  • Line 17: warning: exported function NewRequalifier should have comment or be unexported (golint)
  • Line 23: warning: exported method Requalifier.Typecheck should have comment or be unexported (golint)

• plugins/plugins.go
  • Line 227: warning: exported type ConfigDecoder should have comment or be unexported (golint)
  • Line 231: warning: exported function Run should have comment or be unexported (golint)

• datasources/json/impl.go
  • Line 18: warning: exported function Creator should have comment or be unexported (golint)
  • Line 76: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)

• logical/tvf.go
  • Line 13: warning: exported type TableValuedFunctionDescription should have comment or be unexported (golint)
  • Line 18: warning: exported type TableValuedFunctionTypecheckedArgument should have comment or be unexported (golint)
  • Line 23: warning: exported type TableValuedFunctionDescriptor should have comment or be unexported (golint)
  • Line 30: warning: exported type TableValuedFunctionArgumentMatcher should have comment or be unexported (golint)
  • Line 39: warning: exported type TableValuedFunctionArgumentMatcherExpression should have comment or be unexported (golint)
  • Line 43: warning: exported type TableValuedFunctionArgumentMatcherTable should have comment or be unexported (golint)
  • Line 46: warning: exported type TableValuedFunctionArgumentMatcherDescriptor should have comment or be unexported (golint)
  • Line 49: warning: exported type TableValuedFunctionArgumentValue should have comment or be unexported (golint)
  • Line 57: warning: exported type TableValuedFunctionArgumentValueExpression should have comment or be unexported (golint)
  • Line 61: warning: exported function NewTableValuedFunctionArgumentValueExpression should have comment or be unexported (golint)
  • Line 65: warning: exported method TableValuedFunctionArgumentValueExpression.Typecheck should have comment or be unexported (golint)
  • Line 74: warning: exported type TableValuedFunctionArgumentValueTable should have comment or be unexported (golint)
  • Line 78: warning: exported function NewTableValuedFunctionArgumentValueTable should have comment or be unexported (golint)
  • Line 82: warning: exported method TableValuedFunctionArgumentValueTable.Typecheck should have comment or be unexported (golint)
  • Line 92: warning: exported type TableValuedFunctionArgumentValueDescriptor should have comment or be unexported (golint)
  • Line 96: warning: exported function NewTableValuedFunctionArgumentValueDescriptor should have comment or be unexported (golint)
  • Line 100: warning: exported method TableValuedFunctionArgumentValueDescriptor.Typecheck should have comment or be unexported (golint)
  • Line 113: warning: exported type TableValuedFunction should have comment or be unexported (golint)
  • Line 118: warning: exported function NewTableValuedFunction should have comment or be unexported (golint)
  • Line 122: warning: exported method TableValuedFunction.Typecheck should have comment or be unexported (golint)

• physical/transform.go
  • Line 3: warning: exported type Transformers should have comment or be unexported (golint)
  • Line 8: warning: comment on exported method Transformers.TransformNode should be of the form "TransformNode ..." (golint)
  • Line 200: warning: exported method Transformers.TransformExpr should have comment or be unexported (golint)

• datasources/csv/impl.go
  • Line 17: warning: exported function Creator should have comment or be unexported (golint)

• datasources/json/execution.go
  • Line 11: warning: should not use dot imports (golint)
  • Line 16: warning: exported type DatasourceExecuting should have comment or be unexported (golint)
  • Line 21: warning: exported method DatasourceExecuting.Run should have comment or be unexported (golint)

• outputs/batch/csv_format.go
  • Line 12: warning: exported type CSVFormatter should have comment or be unexported (golint)
  • Line 16: warning: exported function NewCSVFormatter should have comment or be unexported (golint)
  • Line 24: warning: exported method CSVFormatter.SetSchema should have comment or be unexported (golint)
  • Line 40: warning: exported method CSVFormatter.Close should have comment or be unexported (golint)

• outputs/stream/internally_consistent_output_stream_wrapper.go
  • Line 7: warning: should not use dot imports (golint)
  • Line 10: warning: exported type InternallyConsistentOutputStreamWrapper should have comment or be unexported (golint)
  • Line 14: warning: exported method InternallyConsistentOutputStreamWrapper.Run should have comment or be unexported (golint)

• functions/functions.go
  • Line 18: warning: exported function FunctionMap should have comment or be unexported (golint)

• execution/execution.go
  • Line 12: warning: exported type Node should have comment or be unexported (golint)
  • Line 16: warning: exported type ExecutionContext should have comment or be unexported (golint)
  • Line 21: warning: exported method ExecutionContext.WithRecord should have comment or be unexported (golint)
  • Line 28: warning: exported type VariableContext should have comment or be unexported (golint)
  • Line 33: warning: exported method VariableContext.WithRecord should have comment or be unexported (golint)
  • Line 40: warning: exported type ProduceFn should have comment or be unexported (golint)
  • Line 42: warning: exported type ProduceContext should have comment or be unexported (golint)
  • Line 46: warning: exported function ProduceFromExecutionContext should have comment or be unexported (golint)
  • Line 52: warning: exported function ProduceFnApplyContext should have comment or be unexported (golint)
  • Line 58: warning: exported type Record should have comment or be unexported (golint)
  • Line 64: warning: comment on exported function NewRecord should be of the form "NewRecord ..." (golint)
  • Line 93: warning: exported type MetaSendFn should have comment or be unexported (golint)
  • Line 95: warning: exported type MetadataMessage should have comment or be unexported (golint)
  • Line 100: warning: exported type MetadataMessageType should have comment or be unexported (golint)
  • Line 103: warning: exported const MetadataMessageTypeWatermark should have comment (or a comment on this block) or be unexported (golint)
  • Line 106: warning: exported var WatermarkMaxValue should have comment or be unexported (golint)

• telemetry/telemetry.go
  • Line 39: warning: exported function SendTelemetry should have comment or be unexported (golint)

• plugins/repository/repository.go
  • Line 35: warning: exported type RepositoryEntry should have comment or be unexported (golint)
  • Line 63: warning: exported function AddRepository should have comment or be unexported (golint)
  • Line 85: warning: exported function GetRepositories should have comment or be unexported (golint)
  • Line 110: warning: exported type Repository should have comment or be unexported (golint)
  • Line 117: warning: exported function GetRepository should have comment or be unexported (golint)
  • Line 142: warning: exported type Plugin should have comment or be unexported (golint)
  • Line 154: warning: exported type Manifest should have comment or be unexported (golint)
  • Line 161: warning: exported function GetManifest should have comment or be unexported (golint)
  • Line 185: warning: exported method Manifest.GetBinaryDownloadURL should have comment or be unexported (golint)
  • Line 193: warning: exported type Version should have comment or be unexported (golint)

• helpers/graph/graph.go
  • Line 11: warning: exported type Field should have comment or be unexported (golint)
  • Line 15: warning: exported type Child should have comment or be unexported (golint)
  • Line 20: warning: exported type Node should have comment or be unexported (golint)
  • Line 26: warning: exported function NewNode should have comment or be unexported (golint)
  • Line 32: warning: exported method Node.AddField should have comment or be unexported (golint)
  • Line 39: warning: exported method Node.AddChild should have comment or be unexported (golint)
  • Line 46: warning: exported type Visualizer should have comment or be unexported (golint)
  • Line 50: warning: exported function Show should have comment or be unexported (golint)

• optimizer/push_filter_into_stream_join_key.go
  • Line 5: warning: should not use dot imports (golint)
  • Line 8: warning: exported function PushDownFilterPredicatesIntoStreamJoinKey should have comment or be unexported (golint)
  • Line 94: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• optimizer/remove_unused_map_fields.go
  • Line 4: warning: should not use dot imports (golint)
  • Line 7: warning: exported function RemoveUnusedMapFields should have comment or be unexported (golint)

• logs/logs.go
  • Line 11: warning: exported var Output should have comment or be unexported (golint)

• outputs/stream/printer.go
  • Line 9: warning: should not use dot imports (golint)
  • Line 13: warning: exported type Format should have comment or be unexported (golint)
  • Line 19: warning: exported type OutputPrinter should have comment or be unexported (golint)
  • Line 24: warning: exported function NewOutputPrinter should have comment or be unexported (golint)
  • Line 31: warning: exported method OutputPrinter.Run should have comment or be unexported (golint)
  • Line 46: warning: exported type NativeFormat should have comment or be unexported (golint)
  • Line 50: warning: exported function NewNativeFormat should have comment or be unexported (golint)
  • Line 56: warning: exported method NativeFormat.WriteRecord should have comment or be unexported (golint)
  • Line 61: warning: exported method NativeFormat.WriteMeta should have comment or be unexported (golint)
  • Line 66: warning: exported method NativeFormat.Close should have comment or be unexported (golint)

• logical/utils.go
  • Line 259: warning: exported function EqualExpressions should have comment or be unexported (golint)

• table_valued_functions/tumble.go
  • Line 1: warning: don't use an underscore in package name (golint)
  • Line 13: warning: exported var Tumble should have comment or be unexported (golint)

• plugins/executor/executor.go
  • Line 26: warning: exported type Manager should have comment or be unexported (golint)
  • Line 30: warning: exported type PluginExecutor should have comment or be unexported (golint)
  • Line 39: warning: exported method PluginExecutor.RunPlugin should have comment or be unexported (golint)
  • Line 131: warning: exported method PluginExecutor.Close should have comment or be unexported (golint)
  • Line 153: warning: exported type Database should have comment or be unexported (golint)
  • Line 158: warning: exported method Database.ListTables should have comment or be unexported (golint)
  • Line 162: warning: exported method Database.GetTable should have comment or be unexported (golint)
  • Line 181: warning: exported type PhysicalDatasource should have comment or be unexported (golint)
  • Line 187: warning: exported method PhysicalDatasource.PushDownPredicates should have comment or be unexported (golint)
  • Line 238: warning: exported method PhysicalDatasource.Materialize should have comment or be unexported (golint)
  • Line 281: warning: exported type ExecutionDatasource should have comment or be unexported (golint)
  • Line 285: warning: exported method ExecutionDatasource.Run should have comment or be unexported (golint)

• datasources/plugins/available_plugins.go
  • Line 8: warning: should not use dot imports (golint)

• execution/nodes/lookup_join.go
  • Line 6: warning: should not use dot imports (golint)
  • Line 10: warning: exported type LookupJoin should have comment or be unexported (golint)
  • Line 14: warning: exported function NewLookupJoin should have comment or be unexported (golint)
  • Line 21: warning: exported method LookupJoin.Run should have comment or be unexported (golint)

• logical/union_all.go
  • Line 9: warning: exported type UnionAll should have comment or be unexported (golint)
  • Line 13: warning: exported function NewUnionAll should have comment or be unexported (golint)
  • Line 17: warning: exported method UnionAll.Typecheck should have comment or be unexported (golint)

• logical/union_distinct.go
  • Line 9: warning: exported type UnionDistinct should have comment or be unexported (golint)
  • Line 13: warning: exported function NewUnionDistinct should have comment or be unexported (golint)
  • Line 17: warning: exported method UnionDistinct.Typecheck should have comment or be unexported (golint)

• physical/explain.go
  • Line 11: warning: exported function ExplainNode should have comment or be unexported (golint)
  • Line 148: warning: exported function ExplainExpr should have comment or be unexported (golint)

• aggregates/distinct.go
  • Line 14: warning: exported function DistinctAggregateOverloads should have comment or be unexported (golint)
  • Line 27: warning: exported type Distinct should have comment or be unexported (golint)
  • Line 32: warning: exported function NewDistinctPrototype should have comment or be unexported (golint)
  • Line 55: warning: exported method Distinct.Add should have comment or be unexported (golint)
  • Line 83: warning: exported method Distinct.Trigger should have comment or be unexported (golint)

• aggregates/sum.go
  • Line 11: warning: exported var SumOverloads should have comment or be unexported (golint)
  • Line 29: warning: exported type SumInt should have comment or be unexported (golint)
  • Line 33: warning: exported function NewSumIntPrototype should have comment or be unexported (golint)
  • Line 41: warning: exported method SumInt.Add should have comment or be unexported (golint)
  • Line 50: warning: exported method SumInt.Trigger should have comment or be unexported (golint)
  • Line 54: warning: exported type SumFloat should have comment or be unexported (golint)
  • Line 58: warning: exported function NewSumFloatPrototype should have comment or be unexported (golint)
  • Line 66: warning: exported method SumFloat.Add should have comment or be unexported (golint)
  • Line 75: warning: exported method SumFloat.Trigger should have comment or be unexported (golint)
  • Line 79: warning: exported type SumDuration should have comment or be unexported (golint)
  • Line 83: warning: exported function NewSumDurationPrototype should have comment or be unexported (golint)
  • Line 91: warning: exported method SumDuration.Add should have comment or be unexported (golint)
  • Line 100: warning: exported method SumDuration.Trigger should have comment or be unexported (golint)

• datasources/docs/aggregates.go
  • Line 10: warning: should not use dot imports (golint)

• optimizer/push_filter_into_lookup_join_branch.go
  • Line 5: warning: should not use dot imports (golint)
  • Line 8: warning: exported function PushDownFilterPredicatesIntoLookupJoinBranch should have comment or be unexported (golint)
  • Line 89: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• logical/logical.go
  • Line 11: warning: exported type Environment should have comment or be unexported (golint)
  • Line 18: warning: exported method Environment.GetUnique should have comment or be unexported (golint)
  • Line 24: warning: exported type CommonTableExpression should have comment or be unexported (golint)
  • Line 29: warning: exported method Environment.WithRecordUniqueVariableNames should have comment or be unexported (golint)
  • Line 38: warning: exported type VariableMapping should have comment or be unexported (golint)
  • Line 43: warning: exported method VariableMapping.WithRecordMapping should have comment or be unexported (golint)
  • Line 50: warning: exported method VariableMapping.GetUniqueName should have comment or be unexported (golint)
  • Line 62: warning: exported function GetUniqueNameMatchingVariable should have comment or be unexported (golint)
  • Line 71: warning: exported function ReverseMapping should have comment or be unexported (golint)
  • Line 79: warning: exported type Node should have comment or be unexported (golint)
  • Line 84: warning: exported type DataSource should have comment or be unexported (golint)
  • Line 88: warning: exported function NewDataSource should have comment or be unexported (golint)
  • Line 92: warning: exported method DataSource.Typecheck should have comment or be unexported (golint)
  • Line 126: warning: exported type Expression should have comment or be unexported (golint)
  • Line 135: warning: exported type StarExpression should have comment or be unexported (golint)
  • Line 139: warning: exported function NewStarExpression should have comment or be unexported (golint)
  • Line 143: warning: exported method StarExpression.Typecheck should have comment or be unexported (golint)
  • Line 147: warning: exported type Variable should have comment or be unexported (golint)
  • Line 151: warning: exported function NewVariable should have comment or be unexported (golint)
  • Line 155: warning: exported method Variable.Typecheck should have comment or be unexported (golint)
  • Line 181: warning: exported method Variable.FieldName should have comment or be unexported (golint)
  • Line 185: warning: exported type Constant should have comment or be unexported (golint)
  • Line 189: warning: exported function NewConstant should have comment or be unexported (golint)
  • Line 193: warning: exported method Constant.Typecheck should have comment or be unexported (golint)
  • Line 203: warning: exported type Tuple should have comment or be unexported (golint)
  • Line 207: warning: exported function NewTuple should have comment or be unexported (golint)
  • Line 211: warning: exported method Tuple.Typecheck should have comment or be unexported (golint)
  • Line 227: warning: exported type And should have comment or be unexported (golint)
  • Line 231: warning: exported function NewAnd should have comment or be unexported (golint)
  • Line 235: warning: exported method And.Typecheck should have comment or be unexported (golint)
  • Line 255: warning: exported type Or should have comment or be unexported (golint)
  • Line 259: warning: exported function NewOr should have comment or be unexported (golint)
  • Line 263: warning: exported method Or.Typecheck should have comment or be unexported (golint)
  • Line 283: warning: exported type QueryExpression should have comment or be unexported (golint)
  • Line 287: warning: exported function NewQueryExpression should have comment or be unexported (golint)
  • Line 291: warning: exported method QueryExpression.Typecheck should have comment or be unexported (golint)
  • Line 318: warning: exported type Coalesce should have comment or be unexported (golint)
  • Line 322: warning: exported function NewCoalesce should have comment or be unexported (golint)
  • Line 326: warning: exported method Coalesce.Typecheck should have comment or be unexported (golint)
  • Line 350: warning: exported type Cast should have comment or be unexported (golint)
  • Line 355: warning: exported function NewCast should have comment or be unexported (golint)
  • Line 359: warning: exported method Cast.Typecheck should have comment or be unexported (golint)
  • Line 376: warning: exported function TypecheckExpression should have comment or be unexported (golint)

• plugins/internal/plugins/plugins.go
  • Line 14: warning: exported const APILevel should have comment or be unexported (golint)
  • Line 16: warning: exported method Record.ToNativeRecord should have comment or be unexported (golint)
  • Line 28: warning: exported function NativeRecordToProto should have comment or be unexported (golint)
  • Line 40: warning: exported method MetadataMessage.ToNativeMetadataMessage should have comment or be unexported (golint)
  • Line 47: warning: exported function NativeMetadataMessageToProto should have comment or be unexported (golint)
  • Line 54: warning: exported function NativeValueToProto should have comment or be unexported (golint)
  • Line 97: warning: exported method Value.ToNativeValue should have comment or be unexported (golint)
  • Line 139: warning: exported function NativeSchemaToProto should have comment or be unexported (golint)
  • Line 153: warning: exported method Schema.ToNativeSchema should have comment or be unexported (golint)
  • Line 167: warning: exported function NativeTypeToProto should have comment or be unexported (golint)
  • Line 204: warning: exported method Type.ToNativeType should have comment or be unexported (golint)
  • Line 242: warning: exported function NativePhysicalVariableContextToProto should have comment or be unexported (golint)
  • Line 262: warning: exported method PhysicalVariableContext.ToNativePhysicalVariableContext should have comment or be unexported (golint)
  • Line 280: warning: exported function NativeExecutionVariableContextToProto should have comment or be unexported (golint)
  • Line 297: warning: exported method ExecutionVariableContext.ToNativeExecutionVariableContext should have comment or be unexported (golint)

• plugins/manager/manager.go
  • Line 22: warning: exported type PluginManager should have comment or be unexported (golint)
  • Line 26: warning: exported type PluginMetadata should have comment or be unexported (golint)
  • Line 31: warning: exported type Version should have comment or be unexported (golint)
  • Line 35: warning: exported method PluginManager.ListInstalledPlugins should have comment or be unexported (golint)
  • Line 86: warning: exported method PluginManager.GetPluginBinaryPath should have comment or be unexported (golint)
  • Line 113: warning: exported method PluginManager.Install should have comment or be unexported (golint)

• datasources/docs/function_signatures.go
  • Line 9: warning: should not use dot imports (golint)

• execution/nodes/distinct.go
  • Line 8: warning: should not use dot imports (golint)
  • Line 12: warning: exported type Distinct should have comment or be unexported (golint)
  • Line 16: warning: exported function NewDistinct should have comment or be unexported (golint)
  • Line 42: warning: exported method Distinct.Run should have comment or be unexported (golint)

• execution/nodes/map.go
  • Line 6: warning: should not use dot imports (golint)
  • Line 10: warning: exported type Map should have comment or be unexported (golint)
  • Line 15: warning: exported function NewMap should have comment or be unexported (golint)
  • Line 22: warning: exported method Map.Run should have comment or be unexported (golint)

• outputs/batch/table_format.go
  • Line 12: warning: exported type TableFormatter should have comment or be unexported (golint)
  • Line 16: warning: exported function NewTableFormatter should have comment or be unexported (golint)
  • Line 26: warning: exported method TableFormatter.SetSchema should have comment or be unexported (golint)
  • Line 44: warning: exported method TableFormatter.Close should have comment or be unexported (golint)

• octosql/types.go
  • Line 8: warning: exported type TypeID should have comment or be unexported (golint)
  • Line 11: warning: exported const TypeIDNull should have comment (or a comment on this block) or be unexported (golint)
  • Line 25: warning: exported type Type should have comment or be unexported (golint)
  • Line 49: warning: exported type StructField should have comment or be unexported (golint)
  • Line 54: warning: exported type TypeRelation should have comment or be unexported (golint)
  • Line 57: warning: exported const TypeRelationIsnt should have comment (or a comment on this block) or be unexported (golint)
  • Line 62: warning: exported method Type.Equals should have comment or be unexported (golint)
  • Line 66: warning: exported method Type.Is should have comment or be unexported (golint)
  • Line 198: warning: exported var Null should have comment or be unexported (golint)
  • Line 208: warning: exported function TypeSum should have comment or be unexported (golint)
  • Line 290: warning: exported function TypeIntersection should have comment or be unexported (golint)

• execution/group_key.go
  • Line 11: warning: exported type GroupKey should have comment or be unexported (golint)
  • Line 13: warning: exported type GroupKeyIface should have comment or be unexported (golint)
  • Line 17: warning: exported method GroupKey.GetGroupKey should have comment or be unexported (golint)
  • Line 21: warning: exported method GroupKey.Less should have comment or be unexported (golint)

• aggregates/array.go
  • Line 14: warning: exported var ArrayOverloads should have comment or be unexported (golint)
  • Line 23: warning: comment on exported type Array should be of the form "Array ..." (with optional leading article) (golint)
  • Line 28: warning: exported function NewArrayPrototype should have comment or be unexported (golint)
  • Line 50: warning: exported method Array.Add should have comment or be unexported (golint)
  • Line 75: warning: exported method Array.Trigger should have comment or be unexported (golint)

• datasources/plugins/repositories.go
  • Line 8: warning: should not use dot imports (golint)

• optimizer/filter_datasource_pushdown.go
  • Line 5: warning: should not use dot imports (golint)
  • Line 8: warning: exported function PushDownFilterPredicatesToDatasource should have comment or be unexported (golint)
  • Line 63: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• optimizer/mark_unused_datasource_fields.go
  • Line 4: warning: should not use dot imports (golint)
  • Line 7: warning: exported function RemoveUnusedDatasourceFields should have comment or be unexported (golint)

• logical/filter.go
  • Line 10: warning: exported type Filter should have comment or be unexported (golint)
  • Line 15: warning: exported function NewFilter should have comment or be unexported (golint)
  • Line 19: warning: exported method Filter.Typecheck should have comment or be unexported (golint)

• logical/join.go
  • Line 9: warning: exported type JoinType should have comment or be unexported (golint)
  • Line 12: warning: exported const JoinTypeLeft should have comment (or a comment on this block) or be unexported (golint)
  • Line 16: warning: exported type JoinStrategy should have comment or be unexported (golint)
  • Line 19: warning: exported const JoinStrategyUndefined should have comment (or a comment on this block) or be unexported (golint)
  • Line 26: warning: exported type Join should have comment or be unexported (golint)
  • Line 30: warning: exported function NewJoin should have comment or be unexported (golint)
  • Line 37: warning: exported method Join.Typecheck should have comment or be unexported (golint)
  • Line 84: warning: exported type LateralJoin should have comment or be unexported (golint)
  • Line 88: warning: exported function NewLateralJoin should have comment or be unexported (golint)
  • Line 95: warning: exported method LateralJoin.Typecheck should have comment or be unexported (golint)

• execution/expressions.go
  • Line 9: warning: exported const BTreeDefaultDegree should have comment or be unexported (golint)
  • Line 11: warning: exported type Expression should have comment or be unexported (golint)
  • Line 15: warning: exported type Variable should have comment or be unexported (golint)
  • Line 19: warning: exported function NewVariable should have comment or be unexported (golint)
  • Line 26: warning: exported method Variable.Evaluate should have comment or be unexported (golint)
  • Line 34: warning: exported type Constant should have comment or be unexported (golint)
  • Line 38: warning: exported function NewConstant should have comment or be unexported (golint)
  • Line 44: warning: exported method Constant.Evaluate should have comment or be unexported (golint)
  • Line 48: warning: exported type TypeAssertion should have comment or be unexported (golint)
  • Line 53: warning: exported function NewTypeAssertion should have comment or be unexported (golint)
  • Line 60: warning: exported method TypeAssertion.Evaluate should have comment or be unexported (golint)
  • Line 76: warning: exported type Cast should have comment or be unexported (golint)
  • Line 81: warning: exported function NewCast should have comment or be unexported (golint)
  • Line 88: warning: exported method Cast.Evaluate should have comment or be unexported (golint)
  • Line 104: warning: exported type FunctionCall should have comment or be unexported (golint)
  • Line 110: warning: exported function NewFunctionCall should have comment or be unexported (golint)
  • Line 118: warning: exported method FunctionCall.Evaluate should have comment or be unexported (golint)
  • Line 140: warning: exported type And should have comment or be unexported (golint)
  • Line 144: warning: exported function NewAnd should have comment or be unexported (golint)
  • Line 150: warning: exported method And.Evaluate should have comment or be unexported (golint)
  • Line 172: warning: exported type Or should have comment or be unexported (golint)
  • Line 176: warning: exported function NewOr should have comment or be unexported (golint)
  • Line 182: warning: exported method Or.Evaluate should have comment or be unexported (golint)
  • Line 202: warning: exported type SingleColumnQueryExpression should have comment or be unexported (golint)
  • Line 206: warning: exported function NewSingleColumnQueryExpression should have comment or be unexported (golint)
  • Line 212: warning: exported method SingleColumnQueryExpression.Evaluate should have comment or be unexported (golint)
  • Line 229: warning: exported type MultiColumnQueryExpression should have comment or be unexported (golint)
  • Line 233: warning: exported function NewMultiColumnQueryExpression should have comment or be unexported (golint)
  • Line 239: warning: exported method MultiColumnQueryExpression.Evaluate should have comment or be unexported (golint)
  • Line 256: warning: exported type Coalesce should have comment or be unexported (golint)
  • Line 260: warning: exported function NewCoalesce should have comment or be unexported (golint)
  • Line 266: warning: exported method Coalesce.Evaluate should have comment or be unexported (golint)
  • Line 279: warning: exported type Tuple should have comment or be unexported (golint)
  • Line 283: warning: exported function NewTuple should have comment or be unexported (golint)
  • Line 289: warning: exported method Tuple.Evaluate should have comment or be unexported (golint)

• physical/physical.go
  • Line 13: warning: comment on exported type Environment should be of the form "Environment ..." (with optional leading article) (golint)
  • Line 22: warning: exported method Environment.WithRecordSchema should have comment or be unexported (golint)
  • Line 28: warning: exported type VariableContext should have comment or be unexported (golint)
  • Line 33: warning: exported method VariableContext.WithRecordSchema should have comment or be unexported (golint)
  • Line 40: warning: exported type AggregateDetails should have comment or be unexported (golint)
  • Line 45: warning: exported type AggregateDescriptor should have comment or be unexported (golint)
  • Line 52: warning: exported type DatasourceRepository should have comment or be unexported (golint)
  • Line 61: warning: exported type Database should have comment or be unexported (golint)
  • Line 66: warning: exported method DatasourceRepository.GetDatasource should have comment or be unexported (golint)
  • Line 103: warning: exported type DatasourceImplementation should have comment or be unexported (golint)
  • Line 108: warning: exported type FunctionDetails should have comment or be unexported (golint)
  • Line 113: warning: exported type FunctionDescriptor should have comment or be unexported (golint)

• config/config.go
  • Line 24: warning: exported type Config should have comment or be unexported (golint)
  • Line 28: warning: exported type DatabaseConfig should have comment or be unexported (golint)
  • Line 35: warning: exported function Read should have comment or be unexported (golint)
  • Line 54: warning: exported type YamlUnmarshallableVersionConstraint should have comment or be unexported (golint)
  • Line 56: warning: exported method YamlUnmarshallableVersionConstraint.UnmarshalText should have comment or be unexported (golint)
  • Line 65: warning: exported method YamlUnmarshallableVersionConstraint.Raw should have comment or be unexported (golint)
  • Line 69: warning: exported function NewYamlUnmarshallableVersionConstraint should have comment or be unexported (golint)
  • Line 73: warning: exported type PluginReference should have comment or be unexported (golint)
  • Line 78: warning: exported method PluginReference.UnmarshalText should have comment or be unexported (golint)

• datasources/memory/execution.go
  • Line 7: warning: should not use dot imports (golint)
  • Line 10: warning: exported type Datasource should have comment or be unexported (golint)
  • Line 14: warning: exported type Entry should have comment or be unexported (golint)
  • Line 20: warning: exported method Datasource.Run should have comment or be unexported (golint)

• logical/order_by.go
  • Line 10: warning: exported type OrderDirection should have comment or be unexported (golint)
  • Line 12: warning: exported function DirectionsToMultipliers should have comment or be unexported (golint)
  • Line 28: warning: exported type OrderBy should have comment or be unexported (golint)
  • Line 34: warning: exported function NewOrderBy should have comment or be unexported (golint)
  • Line 42: warning: exported method OrderBy.Typecheck should have comment or be unexported (golint)

• logical/with.go
  • Line 9: warning: exported type With should have comment or be unexported (golint)
  • Line 15: warning: exported function NewWith should have comment or be unexported (golint)
  • Line 23: warning: exported method With.Typecheck should have comment or be unexported (golint)

• table_valued_functions/poll.go
  • Line 1: warning: don't use an underscore in package name (golint)
  • Line 14: warning: exported var Poll should have comment or be unexported (golint)

• table_valued_functions/range.go
  • Line 1: warning: don't use an underscore in package name (golint)
  • Line 14: warning: exported var Range should have comment or be unexported (golint)

• physical/expression.go
  • Line 12: warning: exported type Expression should have comment or be unexported (golint)
  • Line 29: warning: exported type ExpressionType should have comment or be unexported (golint)
  • Line 32: warning: exported const ExpressionTypeVariable should have comment (or a comment on this block) or be unexported (golint)
  • Line 70: warning: exported type Variable should have comment or be unexported (golint)
  • Line 75: warning: exported type Constant should have comment or be unexported (golint)
  • Line 79: warning: exported type FunctionCall should have comment or be unexported (golint)
  • Line 85: warning: exported type And should have comment or be unexported (golint)
  • Line 89: warning: exported type Or should have comment or be unexported (golint)
  • Line 93: warning: exported type QueryExpression should have comment or be unexported (golint)
  • Line 97: warning: exported type Coalesce should have comment or be unexported (golint)
  • Line 101: warning: exported type Tuple should have comment or be unexported (golint)
  • Line 105: warning: exported type TypeAssertion should have comment or be unexported (golint)
  • Line 110: warning: exported type Cast should have comment or be unexported (golint)
  • Line 115: warning: exported method Expression.Materialize should have comment or be unexported (golint)
  • Line 184: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 226: warning: exported function VariableNameMatchesField should have comment or be unexported (golint)
  • Line 237: warning: exported method Expression.SplitByAnd should have comment or be unexported (golint)
  • Line 248: warning: exported method Expression.VariablesUsed should have comment or be unexported (golint)

• physical/nodes.go
  • Line 13: warning: exported type Node should have comment or be unexported (golint)
  • Line 31: warning: exported type Schema should have comment or be unexported (golint)
  • Line 37: warning: exported function NewSchema should have comment or be unexported (golint)
  • Line 44: warning: exported type SchemaField should have comment or be unexported (golint)
  • Line 49: warning: exported type NodeType should have comment or be unexported (golint)
  • Line 52: warning: exported const NodeTypeDatasource should have comment (or a comment on this block) or be unexported (golint)
  • Line 93: warning: exported type Datasource should have comment or be unexported (golint)
  • Line 100: warning: exported method Datasource.PushDownPredicates should have comment or be unexported (golint)
  • Line 141: warning: exported type Distinct should have comment or be unexported (golint)
  • Line 145: warning: exported type Filter should have comment or be unexported (golint)
  • Line 150: warning: exported type GroupBy should have comment or be unexported (golint)
  • Line 160: warning: exported type Aggregate should have comment or be unexported (golint)
  • Line 166: warning: exported type StreamJoin should have comment or be unexported (golint)
  • Line 171: warning: exported type LookupJoin should have comment or be unexported (golint)
  • Line 175: warning: exported type Map should have comment or be unexported (golint)
  • Line 180: warning: exported type OrderBy should have comment or be unexported (golint)
  • Line 186: warning: exported type Requalifier should have comment or be unexported (golint)
  • Line 191: warning: exported type TableValuedFunction should have comment or be unexported (golint)
  • Line 197: warning: exported type TableValuedFunctionArgument should have comment or be unexported (golint)
  • Line 217: warning: exported type TableValuedFunctionArgumentType should have comment or be unexported (golint)
  • Line 220: warning: exported const TableValuedFunctionArgumentTypeExpression should have comment (or a comment on this block) or be unexported (golint)
  • Line 225: warning: exported type TableValuedFunctionArgumentExpression should have comment or be unexported (golint)
  • Line 229: warning: exported type TableValuedFunctionArgumentTable should have comment or be unexported (golint)
  • Line 233: warning: exported type TableValuedFunctionArgumentDescriptor should have comment or be unexported (golint)
  • Line 237: warning: exported type TableValuedFunctionDescriptor should have comment or be unexported (golint)
  • Line 241: warning: exported type Unnest should have comment or be unexported (golint)
  • Line 246: warning: exported method Node.Materialize should have comment or be unexported (golint)

• physical/rename_variable.go
  • Line 3: warning: exported function RenameVariables should have comment or be unexported (golint)
  • Line 19: warning: exported function RenameVariablesExpr should have comment or be unexported (golint)

• datasources/csv/execution.go
  • Line 12: warning: should not use dot imports (golint)
  • Line 17: warning: exported type DatasourceExecuting should have comment or be unexported (golint)
  • Line 22: warning: exported method DatasourceExecuting.Run should have comment or be unexported (golint)

• execution/nodes/filter.go
  • Line 6: warning: should not use dot imports (golint)
  • Line 10: warning: exported type Filter should have comment or be unexported (golint)
  • Line 15: warning: exported function NewFilter should have comment or be unexported (golint)
  • Line 22: warning: exported method Filter.Run should have comment or be unexported (golint)

• cmd/describe.go
  • Line 7: warning: should not use dot imports (golint)
  • Line 12: warning: exported var DescribeNodeSchema should have comment or be unexported (golint)
  • Line 30: warning: exported type DescribeNode should have comment or be unexported (golint)
  • Line 34: warning: exported method DescribeNode.Run should have comment or be unexported (golint)

• execution/triggers.go
  • Line 10: warning: exported type Trigger should have comment or be unexported (golint)
  • Line 18: warning: exported type CountingTrigger should have comment or be unexported (golint)
  • Line 26: warning: exported function NewCountingTriggerPrototype should have comment or be unexported (golint)
  • Line 42: warning: exported method CountingTrigger.EndOfStreamReached should have comment or be unexported (golint)
  • Line 46: warning: exported method CountingTrigger.WatermarkReceived should have comment or be unexported (golint)
  • Line 48: warning: exported method CountingTrigger.KeyReceived should have comment or be unexported (golint)
  • Line 69: warning: comment on exported method CountingTrigger.Poll should be of the form "Poll ..." (golint)
  • Line 101: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 106: warning: exported type WatermarkTrigger should have comment or be unexported (golint)
  • Line 116: warning: exported function NewWatermarkTriggerPrototype should have comment or be unexported (golint)
  • Line 128: warning: exported method WatermarkTrigger.EndOfStreamReached should have comment or be unexported (golint)
  • Line 132: warning: exported method WatermarkTrigger.WatermarkReceived should have comment or be unexported (golint)
  • Line 136: warning: comment on exported method WatermarkTrigger.KeyReceived should be of the form "KeyReceived ..." (golint)
  • Line 144: warning: comment on exported method WatermarkTrigger.Poll should be of the form "Poll ..." (golint)
  • Line 183: warning: exported type EndOfStreamTrigger should have comment or be unexported (golint)
  • Line 188: warning: exported function NewEndOfStreamTriggerPrototype should have comment or be unexported (golint)
  • Line 197: warning: exported method EndOfStreamTrigger.EndOfStreamReached should have comment or be unexported (golint)
  • Line 201: warning: exported method EndOfStreamTrigger.WatermarkReceived should have comment or be unexported (golint)
  • Line 203: warning: exported method EndOfStreamTrigger.KeyReceived should have comment or be unexported (golint)
  • Line 207: warning: exported method EndOfStreamTrigger.Poll should have comment or be unexported (golint)
  • Line 225: warning: exported type MultiTrigger should have comment or be unexported (golint)
  • Line 229: warning: exported function NewMultiTriggerPrototype should have comment or be unexported (golint)
  • Line 241: warning: exported method MultiTrigger.EndOfStreamReached should have comment or be unexported (golint)
  • Line 247: warning: exported method MultiTrigger.WatermarkReceived should have comment or be unexported (golint)
  • Line 253: warning: exported method MultiTrigger.KeyReceived should have comment or be unexported (golint)
  • Line 259: warning: exported method MultiTrigger.Poll should have comment or be unexported (golint)

• parser/sqlparser/ast.go
  • Line 476: warning: exported type With should have comment or be unexported (golint)
  • Line 481: warning: exported method With.Format should have comment or be unexported (golint)
  • Line 495: warning: exported type CommonTableExpressions should have comment or be unexported (golint)
  • Line 497: warning: exported method CommonTableExpressions.Format should have comment or be unexported (golint)
  • Line 514: warning: exported type CommonTableExpression should have comment or be unexported (golint)
  • Line 519: warning: exported method CommonTableExpression.Format should have comment or be unexported (golint)
  • Line 2114: warning: exported type TableValuedFunction should have comment or be unexported (golint)
  • Line 2189: warning: exported type ExprTableValuedFunctionArgumentValue should have comment or be unexported (golint)
  • Line 2193: warning: exported method ExprTableValuedFunctionArgumentValue.Format should have comment or be unexported (golint)
  • Line 2207: warning: exported type TableDescriptorTableValuedFunctionArgumentValue should have comment or be unexported (golint)
  • Line 2211: warning: exported method TableDescriptorTableValuedFunctionArgumentValue.Format should have comment or be unexported (golint)
  • Line 2224: warning: exported type FieldDescriptorTableValuedFunctionArgumentValue should have comment or be unexported (golint)
  • Line 2228: warning: exported method FieldDescriptorTableValuedFunctionArgumentValue.Format should have comment or be unexported (golint)
  • Line 3371: warning: exported type ConvertType should have comment or be unexported (golint)
  • Line 3380: warning: exported type ConvertTypeSimple should have comment or be unexported (golint)
  • Line 3393: warning: exported type ConvertTypeList should have comment or be unexported (golint)
  • Line 3412: warning: exported type ConvertTypeObject should have comment or be unexported (golint)
  • Line 3442: warning: exported type ConvertTypeObjectField should have comment or be unexported (golint)
  • Line 3724: warning: exported type Trigger should have comment or be unexported (golint)
  • Line 3734: warning: exported type WatermarkTrigger should have comment or be unexported (golint)
  • Line 3737: warning: exported method WatermarkTrigger.Format should have comment or be unexported (golint)
  • Line 3745: warning: exported type EndOfStreamTrigger should have comment or be unexported (golint)
  • Line 3748: warning: exported method EndOfStreamTrigger.Format should have comment or be unexported (golint)
  • Line 3756: warning: exported type DelayTrigger should have comment or be unexported (golint)
  • Line 3760: warning: exported method DelayTrigger.Format should have comment or be unexported (golint)
  • Line 3768: warning: exported type CountingTrigger should have comment or be unexported (golint)
  • Line 3772: warning: exported method CountingTrigger.Format should have comment or be unexported (golint)

• datasources/plugins/available_versions.go
  • Line 8: warning: should not use dot imports (golint)

• optimizer/optimize.go
  • Line 4: warning: should not use dot imports (golint)
  • Line 19: warning: exported function Optimize should have comment or be unexported (golint)

• execution/nodes/unnest.go
  • Line 6: warning: should not use dot imports (golint)
  • Line 12: warning: exported type Unnest should have comment or be unexported (golint)
  • Line 17: warning: exported function NewUnnest should have comment or be unexported (golint)
  • Line 21: warning: exported method Unnest.Run should have comment or be unexported (golint)

• optimizer/push_filter_into_stream_join_branch.go
  • Line 5: warning: should not use dot imports (golint)
  • Line 8: warning: exported function PushDownFilterPredicatesIntoStreamJoinBranch should have comment or be unexported (golint)
  • Line 116: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• optimizer/remove_unused_groupby_fields.go
  • Line 4: warning: should not use dot imports (golint)
  • Line 7: warning: exported function RemoveUnusedGroupByNonKeyFields should have comment or be unexported (golint)

• logical/distinct.go
  • Line 9: warning: exported type Distinct should have comment or be unexported (golint)
  • Line 13: warning: exported function NewDistinct should have comment or be unexported (golint)
  • Line 17: warning: exported method Distinct.Typecheck should have comment or be unexported (golint)

• logical/group_by.go
  • Line 11: warning: exported type Trigger should have comment or be unexported (golint)
  • Line 15: warning: exported type CountingTrigger should have comment or be unexported (golint)
  • Line 19: warning: exported function NewCountingTrigger should have comment or be unexported (golint)
  • Line 23: warning: exported method CountingTrigger.Typecheck should have comment or be unexported (golint)
  • Line 32: warning: exported type DelayTrigger should have comment or be unexported (golint)
  • Line 36: warning: exported function NewDelayTrigger should have comment or be unexported (golint)
  • Line 40: warning: exported method DelayTrigger.Typecheck should have comment or be unexported (golint)
  • Line 44: warning: exported type WatermarkTrigger should have comment or be unexported (golint)
  • Line 47: warning: exported function NewWatermarkTrigger should have comment or be unexported (golint)
  • Line 51: warning: exported method WatermarkTrigger.Typecheck should have comment or be unexported (golint)
  • Line 63: warning: exported type EndOfStreamTrigger should have comment or be unexported (golint)
  • Line 66: warning: exported function NewEndOfStreamTrigger should have comment or be unexported (golint)
  • Line 70: warning: exported method EndOfStreamTrigger.Typecheck should have comment or be unexported (golint)
  • Line 77: warning: exported type GroupBy should have comment or be unexported (golint)
  • Line 89: warning: exported function NewGroupBy should have comment or be unexported (golint)
  • Line 93: warning: exported method GroupBy.Typecheck should have comment or be unexported (golint)

• plugins/internal/plugins/input.go
  • Line 7: warning: exported type PluginInput should have comment or be unexported (golint)

• table_valued_functions/max_diff_watermark.go
  • Line 1: warning: don't use an underscore in package name (golint)
  • Line 14: warning: exported var MaxDiffWatermark should have comment or be unexported (golint)

• physical/triggers.go
  • Line 9: warning: exported type Trigger should have comment or be unexported (golint)
  • Line 18: warning: exported type TriggerType should have comment or be unexported (golint)
  • Line 21: warning: exported const TriggerTypeCounting should have comment (or a comment on this block) or be unexported (golint)
  • Line 41: warning: exported type CountingTrigger should have comment or be unexported (golint)
  • Line 45: warning: exported type EndOfStreamTrigger should have comment or be unexported (golint)
  • Line 48: warning: exported type WatermarkTrigger should have comment or be unexported (golint)
  • Line 52: warning: exported type MultiTrigger should have comment or be unexported (golint)
  • Line 56: warning: exported method Trigger.Materialize should have comment or be unexported (golint)

• aggregates/average.go
  • Line 11: warning: exported var AverageOverloads should have comment or be unexported (golint)
  • Line 29: warning: exported type AverageInt should have comment or be unexported (golint)
  • Line 34: warning: exported function NewAverageIntPrototype should have comment or be unexported (golint)
  • Line 43: warning: exported method AverageInt.Add should have comment or be unexported (golint)
  • Line 48: warning: exported method AverageInt.Trigger should have comment or be unexported (golint)
  • Line 52: warning: exported type AverageFloat should have comment or be unexported (golint)
  • Line 57: warning: exported function NewAverageFloatPrototype should have comment or be unexported (golint)
  • Line 66: warning: exported method AverageFloat.Add should have comment or be unexported (golint)
  • Line 71: warning: exported method AverageFloat.Trigger should have comment or be unexported (golint)
  • Line 75: warning: exported type AverageDuration should have comment or be unexported (golint)
  • Line 80: warning: exported function NewAverageDurationPrototype should have comment or be unexported (golint)
  • Line 89: warning: exported method AverageDuration.Add should have comment or be unexported (golint)
  • Line 94: warning: exported method AverageDuration.Trigger should have comment or be unexported (golint)

• aggregates/min.go
  • Line 14: warning: exported var MinOverloads should have comment or be unexported (golint)
  • Line 32: warning: exported type Min should have comment or be unexported (golint)
  • Line 36: warning: exported function NewMinPrototype should have comment or be unexported (golint)
  • Line 58: warning: exported method Min.Add should have comment or be unexported (golint)
  • Line 83: warning: exported method Min.Trigger should have comment or be unexported (golint)

• aggregates/table.go
  • Line 7: warning: exported var Aggregates should have comment or be unexported (golint)

• datasources/docs/functions.go
  • Line 9: warning: should not use dot imports (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

misspell97%

Misspell Finds commonly misspelled English words

• physical/physical.go
  • Line 13: warning: "seperate" is a misspelling of "separate" (misspell)

• plugins/repository/repository.go
  • Line 92: warning: "addional" is a misspelling of "additional" (misspell)

• parser/sqlparser/dependency/sqltypes/value.go
  • Line 44: warning: "cannonical" is a misspelling of "canonical" (misspell)
  • Line 52: warning: "cannonical" is a misspelling of "canonical" (misspell)
  • Line 142: warning: "cannonical" is a misspelling of "canonical" (misspell)
  • Line 172: warning: "representaion" is a misspelling of "representation" (misspell)