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

Revert "Support fast Zap logging" #365

Closed
wants to merge 2 commits into from
Closed

Conversation

mh-park
Copy link
Contributor

@mh-park mh-park commented Aug 6, 2018

Reverts #361

mh-park added 2 commits August 6, 2018 13:31
This adds a `zapcore.MarshalLogObject/Array` method to all generated structs and its underlying components, enums, and non-primitive typedefs, implementing Zap marshalers (i.e. [ObjectMarshaler](https://godoc.org/go.uber.org/zap/zapcore#ObjectMarshaler), [ArrayMarshaler](https://godoc.org/go.uber.org/zap/zapcore#ArrayMarshaler)). For typedefs of primitives, the logging is up to the user, simply by casting it down to the root type and using the respective `Add/Append...` method of the Zap encoder.
@mh-park mh-park closed this Aug 6, 2018
@mh-park mh-park deleted the revert-361-minho/zap2 branch August 6, 2018 20:38
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.

1 participant