Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Flatten view.Data #403

Open
rakyll opened this issue Feb 4, 2018 · 2 comments
Open

Flatten view.Data #403

rakyll opened this issue Feb 4, 2018 · 2 comments
Assignees
@rakyll
Labels
exporter P2 stats

Comments

@rakyll
Copy link
Contributor

rakyll commented Feb 4, 2018

ViewData is a quite a complex type with dependencies to instrumentation-oriented types. Flatten the fields and reduce the dependency to Aggregation and View types.
@rakyll rakyll self-assigned this Feb 4, 2018
@semistrict
Copy link
Contributor

Given that we plan on providing a remote API for this, should everything be strings/enums?

@rakyll
Copy link
Contributor Author

rakyll commented Feb 6, 2018

@Ramonza, yes.

@semistrict semistrict changed the title Flatten the ViewData Flatten view.Data Feb 13, 2018
@rakyll rakyll added the P2+ label Apr 18, 2018
@rghetia rghetia added P2 and removed P2+ labels Apr 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rakyll rakyll

Labels
exporter P2 stats
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rakyll @semistrict @rghetia