apache / apache/gravitino

[Subtask] Refactor DorisContainer to avoid unused GenericContainer creation

Open
#10,147 0 comments 0 reactions 1 assignee Claimed by @zhoukangcn View on GitHub
improvement
Dominant language
Java
Stars
3.2k
Forks
935
Avg merge
1d 16h
Merged PRs (30d)
298

Description

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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.