Skip to content

Kmonte/shared backend pr2 backend#574

Closed
kmontemayor2-sc wants to merge 2 commits intomainfrom
kmonte/shared-backend-pr2-backend
Closed

Kmonte/shared backend pr2 backend#574
kmontemayor2-sc wants to merge 2 commits intomainfrom
kmonte/shared-backend-pr2-backend

Conversation

@kmontemayor2-sc
Copy link
Copy Markdown
Collaborator

Scope of work done

Where is the documentation for this feature?: N/A

Did you add automated tests or write a test plan?

Updated Changelog.md? NO

Ready for code review?: NO

@kmontemayor2-sc
Copy link
Copy Markdown
Collaborator Author

/all_test

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 17:19:51UTC : 🔄 E2E Test started.

@ 18:49:56UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 17:19:52UTC : 🔄 Scala Unit Test started.

@ 17:28:36UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 17:19:52UTC : 🔄 Python Unit Test started.

@ 18:35:29UTC : ❌ Workflow failed.
Please check the logs for more details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 17:19:56UTC : 🔄 Integration Test started.

@ 18:43:49UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 17:19:57UTC : 🔄 Lint Test started.

@ 17:28:24UTC : ✅ Workflow completed successfully.

…e types

Move _create_dist_sampler() and _prepare_degree_tensors() from
dist_sampling_producer.py into a new gigl/distributed/utils/dist_sampler.py
module as public functions, along with SamplerInput and SamplerRuntime type
aliases. Update dist_sampling_producer.py to import from the new module.

Add InitSamplingBackendRequest and RegisterBackendRequest frozen dataclasses
to messages.py for the upcoming shared sampling backend.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kmontemayor2-sc kmontemayor2-sc force-pushed the kmonte/shared-backend-pr2-backend branch from 152cfaa to f87e4f3 Compare April 6, 2026 17:28
Introduce SharedDistSamplingBackend, a shared sampling backend that
multiplexes multiple compute-rank channels over a single pool of sampling
workers using fair-queued round-robin scheduling. This replaces the
per-producer model with a per-channel model for graph-store mode.

Includes unit tests for helper functions (batch computation, epoch indices,
worker seed ranges) and basic lifecycle (init, epoch start, describe).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kmontemayor2-sc kmontemayor2-sc force-pushed the kmonte/shared-backend-pr2-backend branch from f87e4f3 to 868773e Compare April 6, 2026 18:48
@kmontemayor2-sc
Copy link
Copy Markdown
Collaborator Author

/all_test

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 18:49:27UTC : 🔄 Lint Test started.

@ 18:56:26UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 18:49:27UTC : 🔄 Scala Unit Test started.

@ 18:57:30UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 18:49:28UTC : 🔄 Python Unit Test started.

@ 19:59:40UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 18:49:30UTC : 🔄 E2E Test started.

@ 20:15:48UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

GiGL Automation

@ 18:49:30UTC : 🔄 Integration Test started.

@ 20:15:54UTC : ✅ Workflow completed successfully.

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