BewareMyPower edited a comment on issue #14673: URL: https://github.com/apache/pulsar/issues/14673#issuecomment-1066654426
It looks like the support for `optional` keyword in Protobuf 3 was introduced from [3.15.0](https://github.com/protocolbuffers/protobuf/releases/tag/v3.15.0). However, it's only a part of tests and the Pulsar C++ library doesn't rely on it. I think we should update the documents for building C++ client from source. For developers, Protobuf 3.15.0 or higher is required. For users, Protobuf 3.4.0 or higher is required, and they should add `-DBUILD_TESTS=OFF` option to CMake command. -- 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: commits-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
