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

full functions overview #998

Merged
merged 1 commit into from
Aug 15, 2018
Merged

full functions overview #998

merged 1 commit into from
Aug 15, 2018

Conversation

Dieterbe
Copy link
Contributor

@Dieterbe Dieterbe commented Aug 15, 2018

@shanson7
Copy link
Collaborator

Would it be worth explicitly using N/A for the functions that don't make sense in the metrictank case?

Copy link
Collaborator

@shanson7 shanson7 left a comment

Choose a reason for hiding this comment

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

Awesome

@Dieterbe
Copy link
Contributor Author

Would it be worth explicitly using N/A for the functions that don't make sense in the metrictank case?

i wanted to do that, but i don't have time now to go review all these functions. maybe later

@Dieterbe Dieterbe merged commit 5f47858 into master Aug 15, 2018
@stivenbb
Copy link
Contributor

Maybe there's some benefit in grouping functions by function type? (i.e. Alias, Combine, Filter, etc.)

@Dieterbe Dieterbe deleted the full-functions-overview branch September 18, 2018 09:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants