Skip to content

Logging: reading from multiple databases concurrently #886

Logging: reading from multiple databases concurrently

Logging: reading from multiple databases concurrently #886

Triggered via pull request December 26, 2024 12:11
Status Failure
Total duration 38s
Artifacts

lint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 9 warnings
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: app/server/datasource/rdbms/clickhouse/sql_formatter_test.go#L27
assignments should only be cuddled with other assignments (wsl)
lint: app/server/datasource/rdbms/datasource_factory.go#L190
only one cuddle assignment allowed before if statement (wsl)
lint
issues found
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/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/postgresql/connection_manager.go#L69
confusing-results: unnamed results of the same type may be confusing, consider using named results (revive)
lint: app/server/datasource/rdbms/ms_sql_server/sql_formatter.go#L85
unused-parameter: parameter 'databaseName' seems to be unused, consider removing or renaming it to match ^_ (revive)
lint: app/server/datasource/rdbms/ms_sql_server/connection.go#L25
confusing-results: unnamed results of the same type may be confusing, consider using named results (revive)
lint: app/server/datasource/rdbms/mysql/sql_formatter.go#L81
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)
lint: app/server/datasource/rdbms/datasource.go#L130
import-shadowing: The name 'errgroup' shadows an import name (revive)