Skip to content

Just turn off the s3_Statistics line with a comment#155

Merged
matyasselmeci merged 1 commit into
PelicanPlatform:mainfrom
matyasselmeci:pr/fix-statistics-log
May 21, 2026
Merged

Just turn off the s3_Statistics line with a comment#155
matyasselmeci merged 1 commit into
PelicanPlatform:mainfrom
matyasselmeci:pr/fix-statistics-log

Conversation

@matyasselmeci

Copy link
Copy Markdown
Contributor

This isn't a 'real' fix for #135 but the current behavior is bad enough that we really need this bandaid.

Comment thread src/S3File.cc

@jhiemstrawisc jhiemstrawisc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. If you haven't already, can you open a follow-up issue for figuring this out?

@matyasselmeci

Copy link
Copy Markdown
Contributor Author

I'll just keep #135 open.

@matyasselmeci matyasselmeci merged commit 8a3d65c into PelicanPlatform:main May 21, 2026
3 checks passed
@matyasselmeci matyasselmeci deleted the pr/fix-statistics-log branch May 21, 2026 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants