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

full functions overview#998

Merged
Dieterbe merged 1 commit intomasterfrom
full-functions-overview
Aug 15, 2018
Merged

full functions overview#998
Dieterbe merged 1 commit intomasterfrom
full-functions-overview

Conversation

@Dieterbe
Copy link
Copy Markdown
Contributor

@Dieterbe Dieterbe commented Aug 15, 2018

@Dieterbe Dieterbe force-pushed the full-functions-overview branch from 4ec64c6 to cad6baf Compare August 15, 2018 12:53
@shanson7
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

3 participants