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

lower the level of logged messages #176

Closed
stefanofornari opened this issue Sep 15, 2023 · 1 comment · Fixed by #183
Closed

lower the level of logged messages #176

stefanofornari opened this issue Sep 15, 2023 · 1 comment · Fixed by #183
Labels
good first issue Good for newcomers

Comments

@stefanofornari
Copy link
Contributor

The following should be logged at lower level than info.

LOGGER.info("using S3WritableByteChannel as write delegate for path '{}'", s3Path.toUri());

LOGGER.info("using S3ReadAheadByteChannel as read delegate for path '{}'", s3Path.toUri());

logger.info("read {} bytes from {}", s3Bytes.length, key);

logger.info("max read ahead fragments '{}' with size '{}' bytes", maxNumberFragments, maxFragmentSize);

@markjschreiber
Copy link
Contributor

Agree, these should be debug.

@markjschreiber markjschreiber added the good first issue Good for newcomers label Sep 18, 2023
ashamw pushed a commit to ashamw/aws-java-nio-spi-for-s3 that referenced this issue Sep 19, 2023
@ashamw ashamw mentioned this issue Sep 19, 2023
markjschreiber pushed a commit that referenced this issue Sep 19, 2023
Co-authored-by: ashamahadev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants