New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[feat] warmup report disk is full or not #2934

Open

Cyber-SiKu wants to merge 1 commit into opencurve:master from Cyber-SiKu:feat/curvefs/cache_full

Contributor

Cyber-SiKu commented Nov 29, 2023

What problem does this PR solve?

Issue Number: #xxx

Problem Summary:

What is changed and how it works?

What's Changed:

How it Works:

Side effects(Breaking backward compatibility? Performance regression?):

Check List

Relevant documentation/comments is changed or added
I acknowledge that all my contributions will be made under the project's license


          [feat] warmup report disk is full or not

bc95eae

Signed-off-by: Cyber-SiKu <Cyber-SiKu@outlook.com>

montaguelhz reviewed

View reviewed changes

Contributor

montaguelhz left a comment

find a typo

curvefs/src/client/warmup/warmup_manager.h

+                          case WarmupStorageErrorType::Full:
+                              return "full";
+                          default:
+                              return "unkown";

Contributor

montaguelhz Dec 6, 2023

unknown

caoxianfei1 added the pending label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment