fix: typos and variable names across multiple files#779
fix: typos and variable names across multiple files#779cxzl25 wants to merge 5 commits intoCurvineIO:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR focuses on improving code clarity and maintainability by fixing typos and correcting a few variable/method names across Rust, Python, Java, and shell test/benchmark code.
Changes:
- Corrected spelling/wording in test assertions, comments, and log/error strings.
- Renamed a few misspelled identifiers (e.g.,
crate_task_fd→create_task_fd,test_batch_writting→test_batch_writing,premit_res→permit_res). - Fixed a misspelled CLI argument value in a Java benchmark test script (
fasle→false).
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| curvine-tests/tests/timeout_test.rs | Fixes a typo in a test comment. |
| curvine-tests/tests/mount_test.rs | Fixes typos in assertion failure messages. |
| curvine-tests/tests/fs_test.rs | Renames a misspelled async test helper function and its call site. |
| curvine-server/src/master/master_metrics.rs | Fixes a typo in a warning log message (with a remaining grammar nit). |
| curvine-server/src/master/job/job_store.rs | Fixes typo in a job-not-found error string. |
| curvine-server/src/master/job/job_context.rs | Fixes typo in a task-not-found error string. |
| curvine-libsdk/python/curvinefs/curvineClient.py | Renames a misspelled local variable used for returned dict. |
| curvine-libsdk/java/src/main/java/io/curvine/bench/NNBenchWithoutMR.java | Fixes typos in comments/Javadoc (with a remaining grammar nit). |
| curvine-fuse/src/session/fuse_mnt.rs | Renames a misspelled private helper method and updates its call site. |
| curvine-client/src/file/fs_reader_buffer.rs | Renames a misspelled local variable (premit_res → permit_res). |
| curvine-client/src/file/curvine_filesystem.rs | Fixes a typo in a comment. |
| build/tests/java-bench.sh | Fixes misspelled false argument in benchmark script (and commented example). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
curvine-libsdk/java/src/main/java/io/curvine/bench/NNBenchWithoutMR.java
Show resolved
Hide resolved
curvine-libsdk/java/src/main/java/io/curvine/bench/NNBenchWithoutMR.java
Show resolved
Hide resolved
curvine-libsdk/java/src/main/java/io/curvine/bench/NNBenchWithoutMR.java
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
curvine-libsdk/java/src/main/java/io/curvine/bench/NNBenchWithoutMR.java
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.