Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Support projection to IPC streams #1095

Closed
jorgecarleitao opened this issue Jun 23, 2022 · 3 comments · Fixed by #1097
Closed

Support projection to IPC streams #1095

jorgecarleitao opened this issue Jun 23, 2022 · 3 comments · Fixed by #1097
Assignees
@joshuataylor
Labels
feature A new feature good first issue Good for newcomers no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog

Comments

@jorgecarleitao
Copy link
Owner

No description provided.

@jorgecarleitao jorgecarleitao added the feature A new feature label Jun 23, 2022
@jorgecarleitao jorgecarleitao mentioned this issue Jun 23, 2022
Closed
@joshuataylor
Copy link
Contributor

Ha! I just came here to create an issue, asking if this was something that could be done.

I've started looking into it.

@jorgecarleitao
Copy link
Owner Author

Let me know if you need help - it seems that we need to accept projection and push it through the functions up to read_next, like we do for FileReader.

@joshuataylor
Copy link
Contributor

Yep, that's what I'm guessing too. It should be fairly straight forward, and pretty much the same implementation as the file reader, also with similar tests.

Good first issue :)

@jorgecarleitao jorgecarleitao added the good first issue Good for newcomers label Jun 23, 2022
This was referenced Jun 23, 2022
Closed
Merged
Merged
@jorgecarleitao jorgecarleitao added the no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog label Jul 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@joshuataylor joshuataylor

Labels
feature A new feature good first issue Good for newcomers no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added support for projections in reading IPC streams joshuataylor/arrow2
2 participants
@joshuataylor @jorgecarleitao