Skip to content

Conversation

MoritzLoewenstein
Copy link

backport of #37610
'sister' pr of #39838

@nodejs-github-bot nodejs-github-bot added fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run. v14.x labels Sep 4, 2021
Copy link
Member

@targos targos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nits to make the code closer to what's on the master branch

@MoritzLoewenstein MoritzLoewenstein force-pushed the fs-writefile-performance-backport branch from d51c40c to 91cad74 Compare September 5, 2021 10:13
@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 5, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 5, 2021
@nodejs-github-bot
Copy link
Collaborator

@targos
Copy link
Member

targos commented Sep 5, 2021

CI with Python 2 is broken: #39990 (comment)

@MoritzLoewenstein MoritzLoewenstein force-pushed the fs-writefile-performance-backport branch from 91cad74 to 03d36bb Compare September 5, 2021 14:07
Increase the write chunk size in fsPromises writeFile
to improve performance.

PR-URL: nodejs#37610
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@targos targos force-pushed the fs-writefile-performance-backport branch from 03d36bb to 9a5e60b Compare September 7, 2021 09:19
@targos
Copy link
Member

targos commented Sep 7, 2021

I just rebased the branch. CI should pass now.

@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 7, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 7, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

targos pushed a commit that referenced this pull request Sep 7, 2021
Increase the write chunk size in fsPromises writeFile
to improve performance.

PR-URL: #37610
Backport-PR-URL: #39993
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@targos
Copy link
Member

targos commented Sep 7, 2021

Landed in d2834fb

@targos targos closed this Sep 7, 2021
@MoritzLoewenstein MoritzLoewenstein deleted the fs-writefile-performance-backport branch September 7, 2021 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

4 participants