Skip to content

support "split by" property #398

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

Open
tomwayson opened this issue Feb 26, 2018 · 0 comments
Open

support "split by" property #398

tomwayson opened this issue Feb 26, 2018 · 0 comments
Labels
Milestone

Comments

@tomwayson
Copy link
Member

Currently the way we group or stack values is by executing multiple queries and joining them. There is another way though. We could instead pass multiple fields to the groupByFieldsForStatistics and make a single query and then convert the response into the same format that is currently the output of the join operation.

This is how Ops Dahsboard does their split by queries and I feel like it should be the default way that cedar does them. For example, would be much nicer if the radar example had 1 query instead of 9.

In order to do this, we'd likely need to add something like splitBy or splitByFields to the definition so we know how to process the query results. We should look at the Ops Dahsboard API (and maybe even source code 😈) for "inspiration."

@tomwayson tomwayson added this to the v1.0.0 RC 1 milestone Feb 26, 2018
@tomwayson tomwayson added the v1x label Feb 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant