nevzheng commented on issue #14153: URL: https://github.com/apache/gravitino/issues/14153#issuecomment-6420675542
### Roadmap and proposed sub-issues Instead of filing sub-issues up front, here is the plan. I'll open each sub-issue when its milestone is ready for review. The draft PRs are on my fork as a stack. | # | Milestone (proposed sub-issue) | Draft PRs | | --- | --- | --- | | 1 | **Proposal (SPIP):** design doc and rationale appendix. Nothing else merges until it's agreed. | nevzheng/gravitino#81 | | 2 | **Standalone wins:** useful without DI, can land during SPIP review. Lombok and Error Prone upgrade, `Lifecycle`, explicit `LockManager`, `CatalogContext`, DI anti-pattern ratchet. | nevzheng/gravitino#77, #79, #80, #92, #82 | | 3 | **Foundation and graph:** Dagger, root `GravitinoComponent` and modules, `ManagedService` start/stop derived from the graph. | nevzheng/gravitino#83, #86, #89, #90 | | 4 | **Seams:** named decorator layers (enum-keyed map, duplicate layer = compile error), HK2 binding from the component, `AuxiliaryContext` for Iceberg REST and Lance, forwarding dispatchers. | nevzheng/gravitino#93, #94, #96, #97, #119 | | 5 | **Host services injection:** server and REST layer take dependencies by constructor. | nevzheng/gravitino#98, #99, #100, #101 | | 6 | **Authorization injection:** authorizer built in the graph, `AuthorizationServices`, read-only `CatalogAccess`. | nevzheng/gravitino#112, #102, #103, #104, #105 | | 7 | **Storage seeds and profiles** out of `GravitinoEnv`. | nevzheng/gravitino#109, #113 | | 8 | **Delete `GravitinoEnv`.** | nevzheng/gravitino#117 | | 9 | **Follow-up:** inject the remaining static utilities and drive `mockStatic` to zero. | later | | 10 | **Follow-up:** per-catalog and per-request scopes where needed. | later | **Order:** 1, with 2 alongside the SPIP review; then 3, 4, then 5 and 6 in parallel, then 7 and 8. #14149 (shutdown fix) lands before 3. The whole migration is meant to land in one release, with no compatibility shims. Feedback on the order or the split is welcome here. -- 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]
