googleapis / googleapis/google-cloud-java

[java-bigquerystorage] Possibly excessive logging for successful stream operations

未关闭
#12,040 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
priority: p2 type: cleanup type: feature request
主要语言
Java
星标
2.1k
派生
1.2k
平均合并
1 天 23 小时
30 天内合并 PR
154

描述

When using bigquerystorage stream writer api, each write requests creates a new `WriterStream` object.

A successful write operation ends with an [RuntimeException](https://github.com/googleapis/google-cloud-java/blob/9f72a6410c1e596dde4d4c7aa5365e1e277b0114/java-bigquerystorage/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1/StreamConnection.java#L94) which is handled by the [StreamWorker](https://github.com/googleapis/google-cloud-java/blob/9f72a6410c1e596dde4d4c7aa5365e1e277b0114/java-bigquerystorage/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1/ConnectionWorker.java#L1391).

Stream worker is printing the "successful error" log messages at an INFO level. Additionally, during product development more log messages in the Stream Worker were moved from `FINE` level into `INFO` level: [PR 1999](https://github.com/googleapis/java-bigquerystorage/pull/1999/changes) and [PR 2017](https://github.com/googleapis/java-bigquerystorage/pull/2017/changes).

As a result of this, every successful write stream now prints quite a lot of what I consider to be non-info level stuff cluttering the logs.

Image

If the maintainers agree that this could be improved, I am willing to contribute updating the log levels as this looks to be a "good first issue" type of problem.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。