refactor(alluxio): reorganize and format alluxio engine internal func…#5789
refactor(alluxio): reorganize and format alluxio engine internal func…#5789RongGu merged 1 commit intofluid-cloudnative:masterfrom
Conversation
…tions Signed-off-by: cheyang <cheyang.cy@alibaba-inc.com>
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: RongGu The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a minor formatting change in pkg/ddc/alluxio/ufs_internal.go, adding a newline between the processUpdatingUFS function and the documentation for updatingUFSWithMountCommand. There are no review comments to address, and I have no feedback to provide.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #5789 +/- ##
=======================================
Coverage 58.46% 58.46%
=======================================
Files 473 473
Lines 32222 32222
=======================================
Hits 18839 18839
Misses 11836 11836
Partials 1547 1547 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
2 participants
…tions
Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?
fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews