chengxilo opened a new issue, #3364:
URL: https://github.com/apache/iggy/issues/3364
### Description
The Go SDK currently uses `log.Printf` for internal logging, which writes
directly to stdout. This is problematic.
Proposed solution (I am not a expert in logger so I believe you can come up
with a better solution):
- Define a minimal logger interface:
```go
type Logger interface {
Debug()
Info()
Error()
Warning()
....
}
```
- Accept it as a client option: `WithLogger(logger Logger)`
- Default to a no-op logger (silent by default), but we still need to have a
default, implemented log system that user can use by simply modify the log
level, for example, set the level to WARNING so we can see the warnings.
- Replace all `log.Printf` calls with c.logger.Printf, mirror the logger
with rust sdk.
### Component
None
### Proposed solution
_No response_
### Alternatives considered
_No response_
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
