Skip to content

Comments

feat(core): implement IoBuf for Buffer#4532

Merged
Xuanwo merged 7 commits intoapache:mainfrom
George-Miao:feat/buffer-io-buf
Apr 25, 2024
Merged

feat(core): implement IoBuf for Buffer#4532
Xuanwo merged 7 commits intoapache:mainfrom
George-Miao:feat/buffer-io-buf

Conversation

@George-Miao
Copy link
Contributor

@George-Miao George-Miao commented Apr 25, 2024

This PR implements compio_buf::IoBuf for Buffer, which is necessary for compfs to implement its accessor. compio_buf::IoVectoredBuf was left unimplemented due to its interface. This is left as a future task.

@George-Miao George-Miao requested a review from Xuanwo as a code owner April 25, 2024 14:42
@github-actions github-actions bot requested review from oowl and silver-ymz April 25, 2024 14:42
@github-actions github-actions bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Apr 25, 2024
@George-Miao George-Miao marked this pull request as draft April 25, 2024 14:45
@George-Miao George-Miao changed the title feat(core): implement compio buf traits for Buffer feat(core): implement IoBuf for Buffer Apr 25, 2024
@George-Miao George-Miao marked this pull request as ready for review April 25, 2024 15:43
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit 95e6a0e into apache:main Apr 25, 2024
@George-Miao George-Miao deleted the feat/buffer-io-buf branch April 25, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants