Skip to content

src: cache continuation_data pointer in MKDirpSync#63335

Open
watilde wants to merge 2 commits into
nodejs:mainfrom
watilde:src/perfomance-improvement
Open

src: cache continuation_data pointer in MKDirpSync#63335
watilde wants to merge 2 commits into
nodejs:mainfrom
watilde:src/perfomance-improvement

Conversation

@watilde
Copy link
Copy Markdown
Member

@watilde watilde commented May 15, 2026

No description provided.

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run. labels May 15, 2026
@watilde watilde force-pushed the src/perfomance-improvement branch from e0b8cdd to 9d973c7 Compare May 15, 2026 15:14
@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

❌ Patch coverage is 89.47368% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.03%. Comparing base (f190a04) to head (3dd89a9).
⚠️ Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
src/node_file.cc 77.77% 0 Missing and 2 partials ⚠️
src/stream_base.cc 93.10% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #63335      +/-   ##
==========================================
- Coverage   90.04%   90.03%   -0.01%     
==========================================
  Files         714      714              
  Lines      225372   225516     +144     
  Branches    42606    42621      +15     
==========================================
+ Hits       202931   203052     +121     
- Misses      14231    14249      +18     
- Partials     8210     8215       +5     
Files with missing lines Coverage Δ
src/node_file.cc 74.19% <77.77%> (ø)
src/stream_base.cc 81.56% <93.10%> (+1.26%) ⬆️

... and 45 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@watilde watilde force-pushed the src/perfomance-improvement branch from f78194a to a027416 Compare May 16, 2026 06:09
Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
@watilde watilde force-pushed the src/perfomance-improvement branch from a027416 to 813b199 Compare May 16, 2026 06:11
Signed-off-by: Daijiro Wachi <daijiro.wachi@gmail.com>
@watilde watilde force-pushed the src/perfomance-improvement branch from 813b199 to 3dd89a9 Compare May 16, 2026 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants