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

Order alias targeting #3196

Closed
nasdf opened this issue Oct 29, 2024 · 0 comments · Fixed by #3217
Closed

Order alias targeting #3196

nasdf opened this issue Oct 29, 2024 · 0 comments · Fixed by #3217
Assignees
Milestone

Comments

@nasdf
Copy link
Member

nasdf commented Oct 29, 2024

Add support for targeting aliased fields in orderings. This does not include aggregates.

@nasdf nasdf self-assigned this Nov 7, 2024
@nasdf nasdf added this to the DefraDB v0.15 milestone Nov 7, 2024
@nasdf nasdf mentioned this issue Nov 7, 2024
4 tasks
@nasdf nasdf closed this as completed in dae2820 Nov 11, 2024
islamaliev pushed a commit to islamaliev/defradb that referenced this issue Nov 13, 2024
## Relevant issue(s)

Resolves sourcenetwork#3196

## Description

This PR adds alias targeting to query orderings.

**Aggregate targets are not included in this PR as they require more
changes.**

## Tasks

- [x] I made sure the code is well commented, particularly
hard-to-understand areas.
- [x] I made sure the repository-held documentation is changed
accordingly.
- [x] I made sure the pull request title adheres to the conventional
commit style (the subset used in the project can be found in
[tools/configs/chglog/config.yml](tools/configs/chglog/config.yml)).
- [x] I made sure to discuss its limitations such as threats to
validity, vulnerability to mistake and misuse, robustness to
invalidation of assumptions, resource requirements, ...

## How has this been tested?

Added integration tests.

Specify the platform(s) on which this was tested:
- MacOS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant