Skip to content

Improvement: Improve filters logging and lessen noise #1586

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 17, 2025
Merged

Improvement: Improve filters logging and lessen noise #1586

merged 2 commits into from
Sep 17, 2025

Conversation

@cstamas
Copy link
Member

@cstamas cstamas commented Sep 14, 2025 •
edited
Loading

Problems:

  • filter logging is not aligned
  • prefix filter logs null for path
  • transfer logs are irrelevant for most user

Changes:

  • align filter logging (both should log all outcomes same way)
  • fix log having null, file is expected to be null at that point
  • prevent transport logs, is just noise

@cstamas
Align filter logging
e90facb 
Also, prefix file logs `null` path, and make both filter show all
cases.
@cstamas cstamas added this to the 2.0.12 milestone Sep 14, 2025
@cstamas cstamas self-assigned this Sep 14, 2025
@cstamas cstamas added the bug Something isn't working label Sep 14, 2025
@cstamas cstamas marked this pull request as ready for review September 14, 2025 21:23
@cstamas cstamas added enhancement New feature or request and removed bug Something isn't working labels Sep 15, 2025
@cstamas cstamas changed the title Bug: Align filter logging, do not log null Improvement: Improve filters logging and lessen noise Sep 15, 2025
@cstamas cstamas requested a review from gnodet September 15, 2025 09:42
@cstamas cstamas merged commit d0daebd into apache:master Sep 17, 2025
8 checks passed
@cstamas cstamas deleted the filter-logging branch September 17, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@slawekjaranowski slawekjaranowski Awaiting requested review from slawekjaranowski

@gnodet gnodet Awaiting requested review from gnodet

Assignees

@cstamas cstamas

Labels

enhancement New feature or request

Projects

None yet

Milestone

2.0.12

Development

Successfully merging this pull request may close these issues.

1 participant

@cstamas