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.

Sign up for GitHub

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

Jump to bottom

Add support for listing all metrics #20

Merged

Licenser merged 2 commits into master from lookup-all

Sep 28, 2016

Contributor

davecromberge commented Sep 28, 2016

Follows from dalmatinerdb/dqe_idx#10


          Add support for listing all metrics

a44e5aa

elvisci reviewed

View reviewed changes

src/query_builder.erl

-                           grouping_where(GroupingNames, 3),
+                           "WHERE " ?MET_TABLE ".collection = $1 ",
+                           MetricWhere,
+                           grouping_where(GroupingNames, N),
                            "AND "],
                   {_N, TagPairs, TagPredicate} =
                       %% We need to multipy count by two since we got names
                       %% and namespcaes
                       build_tag_lookup(Where,3 + GroupingCount * 2),

elvisci Sep 28, 2016

Missing space right "," on line 56


          Fix lint warnings

4e263f0

davecromberge force-pushed the lookup-all branch from 7762adf to 4e263f0 Compare

September 28, 2016 10:10

Licenser merged commit a7f7cc5 into master

davecromberge deleted the lookup-all branch

June 6, 2017 17:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet