kerneltime commented on PR #10473: URL: https://github.com/apache/ozone/pull/10473#issuecomment-4677438227
The 4-PR split is now open and stacked. Each draft PR includes the prior PR's commit on its branch, so reviewers can review them in order: - PR-1 of 4 — Ratis peer addresses as hostnames: #10485 - PR-2 of 4 — Client → OM DNS refresh + shared infrastructure: #10486 (stacked on #10485) - PR-3 of 4 — OM → SCM DNS refresh: #10487 (stacked on #10486) - PR-4 of 4 — DN → SCM heartbeat DNS refresh: #10488 (stacked on #10487) apache/ozone's GitHub config rejects cross-fork base refs, so each PR's GitHub-base is `master` rather than the prior PR's branch. The stacking is encoded in the fork's branch graph: each branch is `git push`-ed off the prior branch's tip, so once PR-1 lands, rebasing PR-2 onto master is `git rebase origin/master` with no conflicts (and the same chain repeats for PR-3 and PR-4). @szetszwo cc — would you like to start on #10485, or have a different review order in mind? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
