Support OData Edm.Stream
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 268
- Forks
- 100
- PR merge metrics
- No merged PRs in 30d
Description
Metadata can expose some property as stream. Imagine PDF file, images or even video files. Python core 'io' module seems like good place to start, especially when working with service returning large files.
odata spec:
https://www.odata.org/blog/enhancing-odata-support-for-streams/
code reference point:
https://github.com/SAP/python-pyodata/blob/3975ed5365b55936872d8e6722545fdf2c481e80/pyodata/v2/service.py#L142
Quick fix for small files seems to be the PR #145 (sample metadata or network log was not provided, so just guessing). But when tried to write the test by myself for this PR and investigated possible usage scenario, found that the problem is bigger.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the OData stream specification and the referenced section of pyodata/v2/service.py around line 142. Review PR #145 and the issue's discussion to understand the proposed small-file fix and why broader support is needed. Done should include a defined, tested approach for Edm.Stream properties, including large files.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100