frederikja163 / frederikja163/Master

Consider adding peekn to GenericReader

Open
#34 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
2
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Consider adding peekn to datacolumnreader. This would probably require modifying its underlying storage to a readonlymemory instead of span. Since we need to modify the data and return an owning copy in the case of big endian readers.

Also would be nice to have a peek of both strings, blobs and units.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating GenericReader and datacolumnreader, then inspect the underlying Span storage and the big endian reader behavior described in the issue. Define the scope for peekn and peeking strings, blobs, and units; done means the supported reader types can return the requested peeked data without the stated ownership problem.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
backend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.