Kangs-Claw opened a new issue, #10147:
URL: https://github.com/apache/gravitino/issues/10147

   ### Describe the subtask
   
   Refactor  to avoid creating an unused  through the  call. The current 
implementation extends  but never uses the parent class's , which adds 
unnecessary overhead and makes the compose-based container indirectly depend on 
BaseContainer's image semantics.
   
   ### Parent issue
   
   #10032
   
   ### Background
   
   In PR #10046, Copilot suggested refactoring  to avoid creating the unused . 
However, this was deferred as it's a larger architectural change beyond the 
scope of the PR.
   
   ### Proposed solution
   
   Consider one of the following approaches:
   1. Remove the  extension and create a compose-specific base/contract
   2. Refactor  to support both single-container and compose-based 
implementations
   3. Create a new  base class
   
   ### Benefits
   
   - Cleaner architecture
   - Reduced overhead
   - Better separation of concerns
   - Easier to maintain and extend


-- 
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]

Reply via email to