googleapis / googleapis/google-cloud-java
[java-bigquerystorage] Possibly excessive logging for successful stream operations
- Lingua principale
- Java
- Stelle
- 2.1k
- Fork
- 1.2k
- Merge medio
- 1g 23h
- PR unite (30g)
- 157
Descrizione
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.
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.
Guida per i contributori
Apri la guida per i contributori
Valutazione
Questa issue non è ancora stata valutata.