Skip to content
Snippets Groups Projects

MultiFlightStream indicate completion in the queue

Merged Jameson Rollins requested to merge jameson.rollins/arrakis-python:mr-queue-complete into main
All threads resolved!

This should be a more robust and less race condition prone way for the threads to indicate they're done with the streams.

Merge request reports

Pipeline #659999 passed

Pipeline passed for 8d58308d on jameson.rollins:mr-queue-complete

Merged by Patrick GodwinPatrick Godwin 5 months ago (Sep 12, 2024 6:49pm UTC)

Loading

Pipeline #660560 passed

Pipeline passed for 174e09fb on main

Test coverage 41.50% from 0 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jameson Rollins added 2 commits

    added 2 commits

    • 346be77b - add Series method to construct time array
    • 0d283047 - MultiFlightStream indicate completion in the queue

    Compare with previous version

  • Patrick Godwin resolved all threads

    resolved all threads

  • Patrick Godwin approved this merge request

    approved this merge request

  • added 1 commit

    • 8d58308d - MultiFlightStream indicate completion in the queue

    Compare with previous version

  • Jameson Rollins reset approvals from @patrick.godwin by pushing to the branch

    reset approvals from @patrick.godwin by pushing to the branch

  • Jameson Rollins resolved all threads

    resolved all threads

  • Jameson Rollins resolved all threads

    resolved all threads

  • Jonathan Hanks approved this merge request

    approved this merge request

  • Patrick Godwin approved this merge request

    approved this merge request

  • Patrick Godwin mentioned in commit 174e09fb

    mentioned in commit 174e09fb

  • Please register or sign in to reply
    Loading