[AMORO-4236] Add Maintenance tab for table process management#4238
[AMORO-4236] Add Maintenance tab for table process management#4238zhangwl9 wants to merge 2 commits into
Conversation
235a52a to
46c8157
Compare
07bd711 to
1271703
Compare
|
Overall: Good design direction, but suggest refining the category taxonomy before merging. The idea of separating table processes into different tabs is great. However, I think Suggested ClassificationInstead of a binary
Additionally, Rationale
Suggested ApproachFor the backend API, I'd recommend defining three For the frontend, there are two pragmatic options:
Either way, the endpoint could be generalized from What do you think? |
Why are the changes needed?
Close #4236.
Brief change log
Introduces Maintenance tab in the table details page to separate maintenance processes (cleanup operations) from optimizing processes.
Key changes:
processCategoryparameter togetOptimizingProcessesInfoAPI to filter by OPTIMIZING or MAINTENANCE category/maintenance-typesto fetch maintenance-specific process typesMixedAndIcebergTableDescriptorto support category-based filtering with proper type separationMaintenance.vuecomponent that wrapsOptimizing.vuewith maintenance-specific configurationprocessCategoryandcancelModalTitleKeyprops toOptimizing.vuefor flexible reuseHow was this patch tested?
Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Documentation