Skip to content

Expose the lower-level load, parse, print, and save operators #3079

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

Merged
merged 10 commits into from
Apr 17, 2023

Conversation

dominiklohmann
Copy link
Member

@dominiklohmann dominiklohmann commented Apr 14, 2023

This exposes the expert operators underlying from … read … and write … to ….

Fixes tenzir/issues#276
Fixes tenzir/issues#254

@dominiklohmann dominiklohmann added the feature New functionality label Apr 14, 2023
@dominiklohmann dominiklohmann changed the base branch from main to topic/dump-to-save April 14, 2023 16:06
Base automatically changed from topic/dump-to-save to main April 15, 2023 08:31
@dominiklohmann dominiklohmann force-pushed the topic/expose-primitives branch from a1227cd to f609605 Compare April 17, 2023 12:50
This introduces `save` as the new dual to the `load` operator. Writing
the documentation for the new operators in #3074 we found the
the verb `dump` to be very unintuitive.
There exist valid use cases for using const-ref data members.
@dominiklohmann dominiklohmann force-pushed the topic/expose-primitives branch from f609605 to 2276394 Compare April 17, 2023 12:51
@dominiklohmann dominiklohmann requested a review from a team April 17, 2023 12:51
@dominiklohmann dominiklohmann marked this pull request as ready for review April 17, 2023 12:52
@jachris jachris self-requested a review April 17, 2023 13:53
@dominiklohmann dominiklohmann merged commit 49d7ccd into main Apr 17, 2023
@dominiklohmann dominiklohmann deleted the topic/expose-primitives branch April 17, 2023 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants