Skip to content

fix: pings were missing requestIds since the last big refactor#362

Merged
deanq merged 9 commits intomainfrom
fix-pings
Oct 5, 2024
Merged

fix: pings were missing requestIds since the last big refactor#362
deanq merged 9 commits intomainfrom
fix-pings

Conversation

@deanq
Copy link
Copy Markdown
Member

@deanq deanq commented Oct 5, 2024

  • Distinguish JobsQueue(asyncio.Queue) and JobsProgress(set) and reference them appropriately
  • Cleaned up JobScaler.process_job --> rp_job.handle_job
  • Graceful cleanup when worker is killed
  • More tests

@deanq deanq merged commit 6a38a5a into main Oct 5, 2024
@deanq deanq deleted the fix-pings branch October 5, 2024 23:04
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