Skip to content

Redact delayed_jobs handler column in query logs - #5513

Open
kathap wants to merge 2 commits into
mainfrom
delayed-jobs-handler-log-adoption
Open

kathap wants to merge 2 commits into
mainfrom
delayed-jobs-handler-log-adoption

Conversation

@kathap

@kathap kathap commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

The serialized handler column is verbose and clutters the statement log. Strip its value from logged delayed_jobs INSERT/UPDATE statements via a Sequel extension, enabled through DB.connect when query logging is on.

  • A short explanation of the proposed change:
    The serialized handler column is verbose and clutters the statement log. Strip its value from logged delayed_jobs INSERT/UPDATE statements.

  • An explanation of the use cases your change solves
    The handler column holds the full serialized job payload, which produces very long, noisy log lines that make the statement log hard to read and inflate log volume. This cleans it up.

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

kathap added 2 commits October 5, 2026 18:41
The serialized handler column is verbose and clutters the statement
log. Strip its value from logged delayed_jobs INSERT/UPDATE statements
via a Sequel extension, enabled through DB.connect when query logging
is on.
@kathap
kathap force-pushed the delayed-jobs-handler-log-adoption branch from 2e31a22 to c3eb168 Compare October 6, 2026 07:41

This branch has not been deployed

No deployments
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.

1 participant