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

feat(source): protobuf field presence #17264

Open
xiangjinwu opened this issue Jun 14, 2024 · 1 comment
Open

feat(source): protobuf field presence #17264

xiangjinwu opened this issue Jun 14, 2024 · 1 comment

Comments

@xiangjinwu
Copy link
Contributor

Just opening an issue to acknowledge this has not been supported yet.

Without field presence (default), missing fields from a message would be filled with their default value (e.g. 0 or '') rather than null. This used to be the common practice of protobuf.

However in some cases, it may be desired to use null for unset fields and only ingest 0/'' when the producer explicitly set them to this value.

This is related to optional and oneof in protobuf specification.

https://protobuf.dev/programming-guides/field_presence/

@github-actions github-actions bot added this to the release-1.10 milestone Jun 14, 2024
@xiangjinwu xiangjinwu removed this from the release-1.10 milestone Jun 14, 2024
Copy link
Contributor

This issue has been open for 60 days with no activity.

If you think it is still relevant today, and needs to be done in the near future, you can comment to update the status, or just manually remove the no-issue-activity label.

You can also confidently close this issue as not planned to keep our backlog clean.
Don't worry if you think the issue is still valuable to continue in the future.
It's searchable and can be reopened when it's time. 😄

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

1 participant