Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

linking: add LoadRaw and LoadPlusRaw functions to LinkSystem. #267

Merged
merged 1 commit into from
Oct 22, 2021

Conversation

warpfork
Copy link
Collaborator

The new LoadRaw and LoadPlusRaw functions allow getting raw byte slices back along with the processed Node, or just the raw byte slice alone.

These behaviors can be useful if you're building an application which both loads the data and processes it, and also needs to pass on the serial form at the same time.

(Also added a great deal of documentation, including to existing functions which were lacking it.)

@warpfork
Copy link
Collaborator Author

warpfork commented Oct 14, 2021

(Note that in the github UI, I've phrased this as currently being "base" on the featureful-storage branch, to get github to show the relevant diff. I would probably prefer to rebase this onto master after landing that branch, though.) Fixed.

Base automatically changed from featureful-storage to master October 14, 2021 18:06
These allow getting raw byte slices back along with the processed Node,
or just the raw byte slice alone.

Also added a great deal of documentation, including to existing
functions which were lacking it.
return nil, ErrLinkingSetup{"could not choose a hasher", err}
}
if lsys.StorageReadOpener == nil {
return nil, ErrLinkingSetup{"no storage configured for reading", io.ErrClosedPipe} // REVIEW: better cause?
Copy link
Member

Choose a reason for hiding this comment

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

REVIEW: nah, this is a good enough cause

Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

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

@warpfork
Copy link
Collaborator Author

With a lack of complaints, okay, merging :)

Note that I still consider the method names a little tentative, and possibly worth iterating on; I'll consider them stable after we start seeing them used, not before. There may be more PRs touching this area in the next smallnum days :)

@warpfork warpfork merged commit ebf675a into master Oct 22, 2021
@warpfork warpfork deleted the raw-loads branch October 22, 2021 09:32
@aschmahmann aschmahmann mentioned this pull request Dec 1, 2021
80 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants