Skip to content

Remove verbose logging statements from artifacts and flow modules#319

Merged
Edwardvaneechoud merged 1 commit intoclaude/add-artifacts-catalog-deT8efrom
claude/remove-debug-logging-BfuUc
Feb 7, 2026
Merged

Remove verbose logging statements from artifacts and flow modules#319
Edwardvaneechoud merged 1 commit intoclaude/add-artifacts-catalog-deT8efrom
claude/remove-debug-logging-BfuUc

Conversation

@Edwardvaneechoud
Copy link
Copy Markdown
Owner

Summary

This PR removes excessive logging statements throughout the codebase, particularly in the artifacts module and flow execution paths. These logs were primarily informational/debug-level messages that cluttered the output without providing significant operational value.

Key Changes

  • artifacts/init.py: Removed logging import and info-level log in get_storage_backend() that logged staging and artifacts root paths
  • artifacts/routes.py: Removed logging import and multiple info/error-level logs from prepare_upload() and finalize_upload() endpoints
  • flowfile/flow_graph.py: Removed debug log statement when sending source_registration_id to kernel
  • routes/routes.py: Consolidated multiple info/warning logs in _run_and_track() into a single debug-level log for source_registration_id resolution
  • shared/artifact_storage.py: Removed logging import and multiple info/error-level logs from SharedFilesystemStorage initialization, prepare_upload(), and finalize_upload() methods

Implementation Details

  • Removed unused logging imports where no other logging calls remain
  • Replaced multiple conditional info/warning logs in source_registration_id resolution with a single consolidated debug statement
  • Preserved the one warning log in _run_and_track() that handles actual exceptions during database queries
  • No functional changes to the code logic; this is purely a logging cleanup effort

https://claude.ai/code/session_01GTiDh2GVX82XXKct1yKq8J

Strip verbose [artifacts], [artifact_storage], and [source_registration_id]
info/error log statements that were added for development troubleshooting.
The resolution logic is preserved; a single logger.debug remains after
source_registration_id resolution. Unused logging imports and logger
setup are also removed where no other logging remains.

https://claude.ai/code/session_01GTiDh2GVX82XXKct1yKq8J
@Edwardvaneechoud Edwardvaneechoud merged commit 108f6bd into claude/add-artifacts-catalog-deT8e Feb 7, 2026
@Edwardvaneechoud Edwardvaneechoud deleted the claude/remove-debug-logging-BfuUc branch February 7, 2026 14:59
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