errose28 commented on PR #9897:
URL: https://github.com/apache/ozone/pull/9897#issuecomment-4058384442
@sodonnel @dombizita @octachoron This PR should be ready for review. I've
completed a re-review of the changes on my end and the CI is still progressing
on my fork. The patch is quite large, but no single file has more than ~150
lines added and it provides the whole picture of how the component version
manager uses the changes to the features/versions.
If you think it would be easier to review in smaller parts I can optionally
split it into three ~500 line PRs:
- Changes to the `ComponentVersion` classes and corresponding tests.
- This would add the `ZDU` version later required by the `LayoutFeature`
classes.
- `LayoutFeature#nextVersion` would be required by the interface but
remain stubbed out until the following PR.
- Changes to the `LayoutFeature` classes and corresponding tests.
- Addition of the new `VersionManager` and its tests to use the previous
changes.
--
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]
