Skip to content

Conversation

@liyafan82
Copy link
Contributor

After supporting 64-bit ArrowBuf, we need to make file/stream IPC work.

This PR also includes some bug fixes and performance improvements.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This has significant performance overhead.

@github-actions
Copy link

@liyafan82
Copy link
Contributor Author

Also add an integration test for VarCharVector, as it is possible that the size of the offset buffer be larger than Integer.MAX_VALUE

@liyafan82 liyafan82 force-pushed the fly_0429_ipc branch 2 times, most recently from ef547e5 to aa10aeb Compare May 26, 2020 08:38
@wesm
Copy link
Member

wesm commented Jun 12, 2020

ping @rymurr and/or @emkornfield

@emkornfield
Copy link
Contributor

Looks reasonable to me. Thanks @liyafan82

pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
After supporting 64-bit ArrowBuf, we need to make file/stream IPC work.

This PR also includes some bug fixes and performance improvements.

Closes apache#7071 from liyafan82/fly_0429_ipc

Authored-by: liyafan82 <fan_li_ya@foxmail.com>
Signed-off-by: Micah Kornfield <emkornfield@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants