Skip to content
This repository has been archived by the owner on Jun 20, 2022. It is now read-only.

Sequence numbers should be part of IPC message custom metadata #26

Open
mbrobbel opened this issue May 25, 2021 · 0 comments
Open

Sequence numbers should be part of IPC message custom metadata #26

mbrobbel opened this issue May 25, 2021 · 0 comments
Assignees
@johanpel
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@mbrobbel
Copy link
Contributor

c4e503c add sequence numbers to schemas, however they are not part of the IPC record batch message. Instead they should be added as custom metadata to the IPC message with:

https://github.com/apache/arrow/blob/3fe2df7b00291752e49beb3ee671a39df9a69cf4/cpp/src/arrow/ipc/writer.cc#L177-L182
@mbrobbel mbrobbel added enhancement New feature or request good first issue Good for newcomers labels May 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@johanpel johanpel

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mbrobbel @johanpel