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

with options should be parsed into a struct #13061

Open
kwannoel opened this issue Oct 26, 2023 · 5 comments
Open

with options should be parsed into a struct #13061

kwannoel opened this issue Oct 26, 2023 · 5 comments

Comments

@kwannoel
Copy link
Contributor

kwannoel commented Oct 26, 2023

https://github.com/risingwavelabs/risingwave/pull/13009/files#r1372493669

@xiangjinwu
Copy link
Contributor

It is already hard to known what's inside with-options today 😢

@kwannoel
Copy link
Contributor Author

It is already hard to known what's inside with-options today 😢

Yeah we should refactor it asap, before the problem gets worse 😆

@BugenZhao
Copy link
Member

This is really a long-standing issue: #7034.

Not sure why it's closed as not planned.

@kwannoel kwannoel self-assigned this Oct 26, 2023
@tabVersion
Copy link
Contributor

tabVersion commented Oct 26, 2023

This is really a long-standing issue: #7034.

Not sure why it's closed as not planned.

Because I found it too hard to push forward

@fuyufjh fuyufjh removed this from the release-1.4 milestone Nov 8, 2023
Copy link
Contributor

This issue has been open for 60 days with no activity. Could you please update the status? Feel free to continue discussion or close as not planned.

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

5 participants