CalvinKirs opened a new pull request, #49163:
URL: https://github.com/apache/doris/pull/49163
## PR Description
This PR introduces a unified storage parameter handling mechanism for
storage-related functionalities, including Export, Table-Valued Functions
(TVF), and Backup. The key objective is to decouple business logic from the
underlying storage implementation, ensuring that the business layer only
focuses on its domain logic while storage parameters manage the interaction
with the storage system.
##Key Changes
### Centralized Storage Parameter Management:
Instead of handling storage configurations in multiple places, storage
parameters now serve as the sole interface between the business logic and the
storage layer. This improves maintainability, avoids scattered storage-related
logic, and simplifies future storage extensions. ### Separation of Concerns:
The business layer is storage-agnostic, meaning it does not need to handle
storage-specific details. Storage parameters are responsible for translating
high-level storage configurations into system-specific settings. ### Phase 1
Implementation:
This PR focuses on integrating Export, TVF, and Backup with the new storage
parameter mechanism. Subsequent PRs will address Broker and additional
components, ensuring a smooth and incremental migration. ##Next Steps
Gradually extend this unified approach to Broker and other storage-dependent
functionalities. Continue refining the abstraction to support future storage
systems with minimal modifications. ## Why This Change?
Enhances maintainability by consolidating storage-related logic. Improves
extensibility for future storage backends. Simplifies business logic, ensuring
a cleaner separation of concerns.
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
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]
