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

refactor stream scan proto, decouple variant specific fields #13871

Open
Tracked by #10225
kwannoel opened this issue Dec 8, 2023 · 1 comment
Open
Tracked by #10225

refactor stream scan proto, decouple variant specific fields #13871

kwannoel opened this issue Dec 8, 2023 · 1 comment
Labels
no-issue-activity type/feature

Comments

@kwannoel
Copy link
Contributor

kwannoel commented Dec 8, 2023 •
edited
Loading

Currently it's a union of all the fields required by different stream scan variants.

We should introduce a new definition which uses oneof to decouple variant specific fields.

This refactor will only be worked on once arrangement backfill's implementation is stabilized.

Suggested by @chenzl25 .
@kwannoel kwannoel changed the title refactor stream scan proto refactor stream scan proto, decouple variant specific fields Dec 8, 2023
@github-actions github-actions bot added this to the release-1.6 milestone Dec 8, 2023
@kwannoel kwannoel mentioned this issue Dec 8, 2023
Merged
10 tasks
@kwannoel kwannoel mentioned this issue Jan 9, 2024
Open
22 tasks
@kwannoel kwannoel removed this from the release-1.6 milestone Jan 9, 2024
@github-actions GitHub Actions
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
Assignees
No one assigned
Labels
no-issue-activity type/feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kwannoel