Skip to content

The postgres tables have fixed names so it is not possible to have multiple queues in one database #17

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
Closed
adam-fowler opened this issue Feb 21, 2025 · 4 comments
Closed

The postgres tables have fixed names so it is not possible to have multiple queues in one database #17

adam-fowler opened this issue Feb 21, 2025 · 4 comments
Assignees
@thoven87
Labels
enhancement New feature or request

Comments

@adam-fowler
Copy link
Member

No description provided.

@thoven87
Copy link
Contributor

thoven87 commented Feb 22, 2025 •
edited
Loading

Would adding a job_queue column fix this instead of creating new table per job queue?

Also, if it's fine with you. Is it ok if we move the swift-jobs tables under a swift_jobs schema?

Queries would be as follow:

SELECT 
      job_id
FROM swift_jobs.queue

@thoven87 thoven87 mentioned this issue Feb 22, 2025
Open
@adam-fowler
Copy link
Member Author

No problem changing the table names. A job queue column is worth thinking about.

@thoven87 thoven87 self-assigned this Feb 23, 2025
@thoven87 thoven87 added the enhancement New feature or request label Feb 23, 2025
@thoven87
Copy link
Contributor

thoven87 commented Feb 23, 2025 •
edited
Loading

Implementations in are in the following PRS #20 hummingbird-project/swift-jobs#67 hummingbird-project/swift-jobs-redis#23

@thoven87
Copy link
Contributor

I suppose this issue can be closed now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thoven87 thoven87

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@adam-fowler @thoven87