Skip to content
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

[Feature Request] Allow for Nested aggregate in TOP #5345

Closed
mvadu opened this issue Jan 13, 2016 · 1 comment
Closed

[Feature Request] Allow for Nested aggregate in TOP #5345

mvadu opened this issue Jan 13, 2016 · 1 comment

Comments

@mvadu
Copy link
Contributor

mvadu commented Jan 13, 2016

Adding onto #1821 and #3930
Currently TOP does not support nested functions. My use case is to have a Top(Sum(<field>),1) or Top(Percentile(<field>,90),1) to show a worst performing transaction. Can we have a new feature on TOP to accept and work on the output of another aggregate function?

It should honor all the group conditions, so below query should give me worst performing transactions be each hour.

SELECT top(percentile(ResponseTime,90),1) FROM "Application_Transaction_Metrics" WHERE "State" =~ /California$/ AND time > 1452623400s and time < now() GROUP BY time(1h), "State", BusinessTransaction

@beckettsean
Copy link
Contributor

This feature request has been rolled into an aggregate issue for all function requests, so that we can close this issue until we are ready to work on it.

You may continue to make comments here. Closing the issue does not mean we are rejecting this idea.

jsternberg added a commit that referenced this issue May 18, 2017
The test cases demonstrate that the following issues are resolved
through some newer features.

Related to #813, #5095, #2467, #5345, and #5150.
stuartcarnie pushed a commit that referenced this issue May 25, 2017
The test cases demonstrate that the following issues are resolved
through some newer features.

Related to #813, #5095, #2467, #5345, and #5150.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants