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

IFS to simplify CASE WHEN #20513

Closed
wangsimo0 opened this issue Mar 29, 2023 · 7 comments · May be fixed by #27467
Closed

IFS to simplify CASE WHEN #20513

wangsimo0 opened this issue Mar 29, 2023 · 7 comments · May be fixed by #27467

Comments

@wangsimo0
Copy link
Contributor

In StarRocks, we can only use case when. However, ifs is a more simple way to express case when and widely used in excel, it's very easy to use.

so here is a proposal:

IFS(condition1, then [,condition2, then2, ....], else)
@srikker
Copy link
Contributor

srikker commented Apr 7, 2023

Hi @wangsimo0 , please assign this to me. Thanks!

@happut
Copy link
Contributor

happut commented Apr 25, 2023

Hi @kateshaowanjou @wangsimo0 , Has this issue been assigned? Could you please assign this to me?
Thanks!

@kateshaowanjou
Copy link
Contributor

@happut Sorry, I just found out @srikker also wants to take this issue. Since he asked first, I assigned this one to him. Can you pick another one? 🙏🏻

@srikker
Copy link
Contributor

srikker commented May 6, 2023

This function just make a grammatical transformation or need reImplement? It looks exactly like the "case when" expression.

@LiShuMing
Copy link
Contributor

Yep. I think this can be covered by CaseWhenExpr totally. So just transform to CaseExprr in FE?

@srikker
Copy link
Contributor

srikker commented May 6, 2023

Yep. I think this can be covered by CaseWhenExpr totally. So just transform to CaseExprr in FE?

ok

@wangsimo0 wangsimo0 added the good first issue Good for newcomers label May 12, 2023
Copy link

We have marked this issue as stale because it has been inactive for 6 months. If this issue is still relevant, removing the stale label or adding a comment will keep it active. Otherwise, we'll close it in 10 days to keep the issue queue tidy. Thank you for your contribution to StarRocks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants