Skip to content

Update docs: updated readFile version for async await#1117

Open
swifty-amay wants to merge 2 commits intovapor:mainfrom
swifty-amay:my-doc-update
Open

Update docs: updated readFile version for async await#1117
swifty-amay wants to merge 2 commits intovapor:mainfrom
swifty-amay:my-doc-update

Conversation

@swifty-amay
Copy link
Copy Markdown

###Changes

  • Updated the depricated version of readFile method for reading files asynchronously.

  • Changes
    try await req.fileio.readFile(at: "/path/to/file") { chunk in
    print(chunk) // ByteBuffer
    }
    (Depricated version)

    to

    let file = try await request.fileio.readFile(at: path, chunkSize: 16 * 1024) // 32Kb, ~5 chunks

@swifty-amay swifty-amay requested review from 0xTim and gwynne as code owners April 8, 2026 17:00
Copy link
Copy Markdown
Member

@ptoffy ptoffy left a comment

Choose a reason for hiding this comment

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

Thanks for this! We can expand it to loop over the chunks AsyncSequence which will be what most users will do

@swifty-amay
Copy link
Copy Markdown
Author

swifty-amay commented Apr 8, 2026

Thanks for the review, actually you're right, devs must also get to know how to loop over the chunks.

@TheHandyOwl TheHandyOwl added the translation-update This PR contains translations for the docs label Apr 11, 2026
Copy link
Copy Markdown
Member

@ptoffy ptoffy left a comment

Choose a reason for hiding this comment

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

readContent is undefined here. Let's just keep the behaviour we had before where we simply print the ByteBuffer chunk

@swifty-amay
Copy link
Copy Markdown
Author

Yes, you're right. Well, let me see if I can do something with this readContent, if not then I'll update this to the previous version where the chunks were being printed directly.

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

Labels

translation-update This PR contains translations for the docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants