-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Support extra output field in TOP function #135434
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
base: main
Are you sure you want to change the base?
Conversation
ebada8e to
d7003ba
Compare
ℹ️ Important: Docs version tagging👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version. We use applies_to tags to mark version-specific features and changes. Expand for a quick overviewWhen to use applies_to tags:✅ At the page level to indicate which products/deployments the content applies to (mandatory) What NOT to do:❌ Don't remove or replace information that applies to an older version 🤔 Need help?
|
8995f31 to
f76d6e7
Compare
8cc245c to
0e61859
Compare
0e61859 to
5b005df
Compare
5b476e4 to
5b005df
Compare
225893a to
a5e475e
Compare
ae44028 to
7c1418b
Compare
cbc8827 to
6c1ccd2
Compare
Done, PTAL |
- Fix grow() method in X-BucketedSort - Add `date` type to supported types of `outputField`
d0c85d1 to
2fe8d31
Compare
757567d to
bd439fe
Compare
There are cases where we would like to answer queries like:
"give me list of employees sorted by their salaries"
"give me list of countries sorted by their area"
etc.
Currently, the
TOPaggregation function only supports one field which is used as both sort field and the output field.This PR enhances the
TOPfunction by adding an optional parametermapToFieldwhich is the field we want as the output. The sorting will still be performed on thefield, just like today.Example:
"give me salaries of 3 youngest employees by gender"
Fixes #128630