Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

YQ-2371: use pure zap logger #13

Merged
merged 1 commit into from
Dec 25, 2023

YQ-2371: use pure zap logger

9700d25
Select commit
Loading
Failed to load commit list.
Merged

YQ-2371: use pure zap logger #13

YQ-2371: use pure zap logger
9700d25
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 25, 2023 in 0s

22.22% of diff hit (target 17.81%)

View this Pull Request on Codecov

22.22% of diff hit (target 17.81%)

Annotations

Check warning on line 59 in app/server/datasource/rdbms/clickhouse/connection_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/datasource/rdbms/clickhouse/connection_manager.go#L59

Added line #L59 was not covered by tests

Check warning on line 106 in app/server/datasource/rdbms/clickhouse/connection_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/datasource/rdbms/clickhouse/connection_manager.go#L106

Added line #L106 was not covered by tests

Check warning on line 141 in app/server/datasource/rdbms/clickhouse/connection_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/datasource/rdbms/clickhouse/connection_manager.go#L141

Added line #L141 was not covered by tests

Check warning on line 119 in app/server/datasource/rdbms/postgresql/connection_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/datasource/rdbms/postgresql/connection_manager.go#L119

Added line #L119 was not covered by tests

Check warning on line 59 in app/server/datasource/rdbms/ydb/connection_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/datasource/rdbms/ydb/connection_manager.go#L59

Added line #L59 was not covered by tests

Check warning on line 112 in app/server/datasource/rdbms/ydb/connection_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/datasource/rdbms/ydb/connection_manager.go#L112

Added line #L112 was not covered by tests

Check warning on line 97 in app/server/utils/errors.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/errors.go#L94-L97

Added lines #L94 - L97 were not covered by tests

Check warning on line 19 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L18-L19

Added lines #L18 - L19 were not covered by tests

Check warning on line 28 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L22-L28

Added lines #L22 - L28 were not covered by tests

Check warning on line 38 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L38

Added line #L38 was not covered by tests

Check warning on line 42 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L42

Added line #L42 was not covered by tests

Check warning on line 47 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L47

Added line #L47 was not covered by tests

Check warning on line 50 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L50

Added line #L50 was not covered by tests

Check warning on line 58 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L58

Added line #L58 was not covered by tests

Check warning on line 62 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L61-L62

Added lines #L61 - L62 were not covered by tests

Check warning on line 78 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L78

Added line #L78 was not covered by tests

Check warning on line 89 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L85-L89

Added lines #L85 - L89 were not covered by tests

Check warning on line 105 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L105

Added line #L105 was not covered by tests

Check warning on line 119 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L119

Added line #L119 was not covered by tests

Check warning on line 121 in app/server/utils/logger.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/server/utils/logger.go#L121

Added line #L121 was not covered by tests