Skip to content

fix: use configured poll_strategy in PgPollFetcher - #112

Open
senysenyseny16 wants to merge 1 commit into
apalis-dev:mainfrom
senysenyseny16:fix/use-configured-poll-strategy
Open

fix: use configured poll_strategy in PgPollFetcher#112
senysenyseny16 wants to merge 1 commit into
apalis-dev:mainfrom
senysenyseny16:fix/use-configured-poll-strategy

Conversation

@senysenyseny16

Copy link
Copy Markdown

Hi, nice project, thank you for your work!

Description

I encountered the following problem:

PgPollFetcher ignored Config::poll_strategy entirely and used a
hardcoded 1s→5min exponential backoff. Config::with_poll_interval
had no effect on the Postgres backend.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Refactoring (no functional changes)

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • I have run the existing tests and they pass
  • I have run cargo fmt and cargo clippy

Checklist

  • My code follows the code style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

If something is wrong, I will be happy to correct the mistakes.

@senysenyseny16
senysenyseny16 force-pushed the fix/use-configured-poll-strategy branch 2 times, most recently from 5132469 to a5e7c23 Compare September 4, 2026 15:00
@senysenyseny16
senysenyseny16 force-pushed the fix/use-configured-poll-strategy branch from a5e7c23 to 8fb48aa Compare September 4, 2026 15:50
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