Skip to content

Logging: reading from multiple databases concurrently #884

Logging: reading from multiple databases concurrently

Logging: reading from multiple databases concurrently #884

Triggered via pull request December 26, 2024 10:36
Status Failure
Total duration 54s
Artifacts

lint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 8 warnings
lint: app/server/datasource/rdbms/datasource.go#L1
: # github.com/ydb-platform/fq-connector-go/app/server/datasource/rdbms [github.com/ydb-platform/fq-connector-go/app/server/datasource/rdbms.test]
lint: app/server/datasource/rdbms/datasource_test.go#L103
cannot use sink (variable of type *paging.SinkMock) as *paging.SinkFactory[any] value in argument to dataSource.ReadSplit
lint: app/server/datasource/rdbms/datasource_test.go#L159
cannot use sink (variable of type *paging.SinkMock) as *paging.SinkFactory[any] value in argument to datasource.ReadSplit (typecheck)
lint: app/server/streaming/doc.go#L1
: # github.com/ydb-platform/fq-connector-go/app/server/streaming [github.com/ydb-platform/fq-connector-go/app/server/streaming.test]
lint: app/server/streaming/streamer_test.go#L257
undefined: paging.NewSink (typecheck)
lint: app/server/paging/sink_factory.go#L6
File is not `goimports`-ed with -local github.com/ydb-platform/fq-connector-go (goimports)
lint: app/server/datasource/rdbms/logging/resolver.go#L6
File is not `goimports`-ed with -local github.com/ydb-platform/fq-connector-go (goimports)
lint: app/server/datasource/rdbms/logging/resolver_dynamic.go#L7
File is not `goimports`-ed with -local github.com/ydb-platform/fq-connector-go (goimports)
lint: app/server/paging/sink_factory.go#L94
ranges should only be cuddled with assignments used in the iteration (wsl)
lint: app/server/paging/sink_factory.go#L123
return statements should not be cuddled if block has more than two lines (wsl)
lint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
lint: app/server/datasource/rdbms/utils/sql_mock.go#L33
confusing-results: unnamed results of the same type may be confusing, consider using named results (revive)
lint: app/server/datasource/rdbms/clickhouse/connection_http.go#L75
confusing-results: unnamed results of the same type may be confusing, consider using named results (revive)
lint: app/server/datasource/rdbms/clickhouse/sql_formatter.go#L86
unused-parameter: parameter 'databaseName' seems to be unused, consider removing or renaming it to match ^_ (revive)
lint: app/server/datasource/rdbms/clickhouse/connection_native.go#L77
confusing-results: unnamed results of the same type may be confusing, consider using named results (revive)
lint: app/server/datasource/rdbms/oracle/sql_formatter.go#L86
unused-parameter: parameter 'databaseName' seems to be unused, consider removing or renaming it to match ^_ (revive)
lint: app/server/datasource/rdbms/postgresql/sql_formatter.go#L89
unused-parameter: parameter 'databaseName' seems to be unused, consider removing or renaming it to match ^_ (revive)
lint: app/server/datasource/rdbms/logging/resolver_static.go#L51
unused-receiver: method receiver 'r' is not referenced in method's body, consider removing or renaming it to match ^_ (revive)