Skip to content

Rename the remaining cell start and stop wording to add and dispose - #2070

Open
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-82from
tom/refactor-miles/op13-84
Open

Rename the remaining cell start and stop wording to add and dispose#2070
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-82from
tom/refactor-miles/op13-84

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

Part of #1837

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 38dd2cf to 611ed8c Compare August 2, 2026 09:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch 2 times, most recently from f844581 to cc6b604 Compare August 2, 2026 09:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch 2 times, most recently from a39737b to 8fc2cf2 Compare August 2, 2026 10:02
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from cc6b604 to 0f1c856 Compare August 2, 2026 10:02
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 8fc2cf2 to e7ed9be Compare August 2, 2026 12:41
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 0f1c856 to 660c384 Compare August 2, 2026 12:41
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from e7ed9be to 98a97cf Compare August 3, 2026 23:44
@fzyzcjy
fzyzcjy requested a review from Rockdu as a code owner August 3, 2026 23:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 660c384 to 5906737 Compare August 3, 2026 23:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 98a97cf to cae3d4a Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 5906737 to 06e2e40 Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from cae3d4a to 66ccbf1 Compare August 4, 2026 06:55
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch 2 times, most recently from 06e2e40 to 4a3d5de Compare August 4, 2026 06:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 66ccbf1 to ef9c0b7 Compare August 4, 2026 07:28
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 4a3d5de to 5b9d878 Compare August 4, 2026 07:28
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from ef9c0b7 to 02d4114 Compare August 4, 2026 07:35
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch 2 times, most recently from 8837ee5 to d01720a Compare August 6, 2026 03:49
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 56c5dc7 to a0c161a Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from fd181c3 to 9609dcc Compare August 10, 2026 06:05
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from a0c161a to fce01bf Compare August 10, 2026 06:05
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 9609dcc to 765db35 Compare August 10, 2026 16:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from fce01bf to 4b6cde8 Compare August 10, 2026 16:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 765db35 to 2c3853e Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch 2 times, most recently from 108a6ea to ea170ae Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 2c3853e to ea06df8 Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch 2 times, most recently from 942463f to 6d46602 Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 2a107f6 to d568b31 Compare August 12, 2026 14:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 6d46602 to 8d8f7bf Compare August 12, 2026 14:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from d568b31 to 974c52d Compare August 12, 2026 23:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch 2 times, most recently from 924544f to ba5495b Compare August 13, 2026 01:13
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from 974c52d to b6cd643 Compare August 13, 2026 01:13
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch 2 times, most recently from f72ff93 to c983893 Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch 2 times, most recently from bd8a5be to dae9e8a Compare August 13, 2026 06:00
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from c983893 to a75abb7 Compare August 13, 2026 06:01
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from dae9e8a to ea9eded Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from a75abb7 to 82bd6f0 Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from ea9eded to f34b9a1 Compare August 14, 2026 04:45
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 82bd6f0 to 49c0176 Compare August 14, 2026 04:46

@guapisolo guapisolo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Codex stack review: #2070 preserves behavior while aligning lifecycle names with ownership—RayWorkerManager owns worker processes, while RolloutServer and ServerCell manage logical membership and router registration. No #2070 follow-up is needed before safe-merge checkpoint #2176; checked tip is #2649.

@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-82 branch from f34b9a1 to a610b52 Compare August 29, 2026 12:38
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-84 branch from 49c0176 to 6b4155b Compare August 29, 2026 12:38
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