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.

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

As a user, I would like to group by "value" instead of just edges. #5412

Closed
MichelDiz opened this issue May 10, 2020 · 3 comments
Closed

As a user, I would like to group by "value" instead of just edges. #5412

MichelDiz opened this issue May 10, 2020 · 3 comments
Labels
area/querylang/groupby Issues related to the @groupby directive. Check if resolved cherry-pick dgraph Issue or PR created by an internal Dgraph contributor. kind/enhancement Something could be better. Stale status/needs-attention This issue needs more eyes on it, more investigation might be required before accepting/rejecting it v22.0.0 Issues related to v22.0.0

Comments

@MichelDiz
Copy link
Contributor

MichelDiz commented May 10, 2020 •
edited
Loading

Experience Report

This issue can be solved by #5335 if we don't wanna do any specific approach for it some how.

We wanna see if this topic is popular or not. Please, share your use cases and "vote" this.

What you wanted to do

Run the groupby func in value edges.

https://dgraph.io/docs/query-language/#groupby

What you actually did

See https://discuss.dgraph.io/t/suggestions-to-improve-groupby/4630

Why that wasn't great, with examples

See https://discuss.dgraph.io/t/suggestions-to-improve-groupby/4630

Any external references to support your case

This issue is similar but it is related to LIST TYPE #4170
See this questions https://discuss.dgraph.io/t/how-to-groupby-date/3040
There are other questions which I don't remember the link, but are several.
@MichelDiz MichelDiz added kind/enhancement Something could be better. status/needs-attention This issue needs more eyes on it, more investigation might be required before accepting/rejecting it area/querylang/groupby Issues related to the @groupby directive. labels May 10, 2020
@MichelDiz MichelDiz reopened this Aug 24, 2022
@MichelDiz
Copy link
Contributor Author

This was partially fixed by #7746
But needs improvement

@MichelDiz MichelDiz added cherry-pick dgraph Issue or PR created by an internal Dgraph contributor. v22.0.0 Issues related to v22.0.0 labels Oct 25, 2022
@rderbier rderbier mentioned this issue Dec 20, 2022
Closed
@MichelDiz
Copy link
Contributor Author

MichelDiz commented Dec 21, 2022 •
edited
Loading

I have added a Cherry-pick PR #8550 that would be a potential fix for this issue.

I will wait the PR be merged to test if it address this issue completely.

@hypermodeinc hypermodeinc deleted a comment from minhaj-shakeel Jan 9, 2023
@MichelDiz MichelDiz moved this from 🆕 In review to 📋 TRIAGE in Feature Requests - PM RoadMap Jan 9, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 3, 2024

This issue has been stale for 60 days and will be closed automatically in 7 days. Comment to keep it open.

@github-actions github-actions bot added the Stale label Aug 3, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/querylang/groupby Issues related to the @groupby directive. Check if resolved cherry-pick dgraph Issue or PR created by an internal Dgraph contributor. kind/enhancement Something could be better. Stale status/needs-attention This issue needs more eyes on it, more investigation might be required before accepting/rejecting it v22.0.0 Issues related to v22.0.0
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MichelDiz @minhaj-shakeel