ikstewa commented on PR #24035: URL: https://github.com/apache/flink/pull/24035#issuecomment-2571733258
@sharath1709 I'm trying to follow up on this change to understand why we do not support optional primitive types. I believe proto3 has support for field presence on primitives when they have the `optional` keyword. Do you have more details on _why_ we specifically do not support primitives? > Additionally, We need to be careful to check for field presence only on non-primitive types if ReadDefaultValues is false and version used is Proto3. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
