igalshilman opened a new pull request #194: URL: https://github.com/apache/flink-statefun/pull/194
Currently the `request-reply.proto` file is located in the `statefun-flink-core` artifact. This file is also used to manually generate the Python SDK `request_reply_pb2.py` file. Since we are about to add additional SDKS and expand the number of proto files, we need to standardize the way we share the SDK proto files. This PR: 1) Adds a maven artifact `statefun-sdk-protos` to hold the request-reply protocol files. This is both a source artifact and a regular jar. Both artifacts contains the same *proto files. TBDs/followups: a) what will we want to publish to maven central? b) The name of the artifact can be something else. c) This PR only deals with the request-reply.proto file, but next we should also add the egresses, and the new types. d) The protobuf package names didn't change yet to reduce to size of the diff, but I think it should happen next. 2) Add a build step to `statefun-flink-core` to build protobufs from that new dependency. 3) Modify the Python SDK build script to include a protobuf generation step. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
