Skip to content

Conversation

@danfimov
Copy link
Owner

@danfimov danfimov commented Oct 9, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 9, 2025 20:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds schedule source functionality for task scheduling in PostgreSQL-backed TaskIQ implementations, introducing AsyncpgScheduleSource for persistent task scheduling and updating project documentation.

  • Implements schedule source class with PostgreSQL storage for task schedules
  • Adds documentation for schedule sources and common issues
  • Updates broker validation logic and removes unused assignment

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/taskiq_pg/asyncpg/schedule_source.py New schedule source implementation with database operations
src/taskiq_pg/asyncpg/queries.py Added SQL queries for schedule table operations
src/taskiq_pg/asyncpg/broker.py Fixed broker startup validation logic
src/taskiq_pg/psqlpy/broker.py Fixed broker startup validation logic
src/taskiq_pg/asyncpg/result_backend.py Removed unnecessary assignment of deletion result
examples/example_with_schedule_source.py Added example demonstrating schedule source usage
docs/ Updated documentation with schedule source tutorial and common issues

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@danfimov danfimov force-pushed the add-schedule-source branch from a95e079 to 1117c67 Compare October 9, 2025 20:40
@danfimov danfimov merged commit 618cfcd into main Oct 9, 2025
4 checks passed
@danfimov danfimov deleted the add-schedule-source branch October 9, 2025 20:43
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