florianvazelle commented on PR #28059:
URL: https://github.com/apache/flink/pull/28059#issuecomment-4603905250

   Hi @khmelevskiy, I'd also like to see this pull request merged.
   
   If you have time, please feel free to take a look at my PR. I don't know if 
it will help the committers speed up the review process, but it's my first 
contribution to the project.
   
   For another topic, do you know, if we don’t use the Protobuf format for the 
API table 
(https://nightlies.apache.org/flink/flink-docs-release-2.2/docs/connectors/table/formats/protobuf/),
 we still need Protobuf? Or does Flink/Beam use it internally, or for something 
else?
   
   If so, perhaps the dependency should be made optional. And a workaround, if 
you’re using uv, would be to use the [override 
dependencies](https://docs.astral.sh/uv/reference/settings/#override-dependencies).
 :thinking: 


-- 
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]

Reply via email to