BewareMyPower commented on code in PR #174:
URL:
https://github.com/apache/pulsar-client-python/pull/174#discussion_r1424719712
##########
setup.py:
##########
@@ -80,7 +80,7 @@ def build_extension(self, ext):
extras_require["functions"] = sorted(
{
"protobuf>=3.6.1,<=3.20.3",
- "grpcio>=1.8.2",
+ "grpcio>=1.60.0",
Review Comment:
I added a workflow to verify the installation in CI. It's not like Maven
because sometimes the Python wheel uses the underlying C extension (like the
Pulsar Python C++ client) and a specific version of dependency A might not be
compatible with dependency B due to the symbols conflict or something else.
However, if all dependencies are compatible, using the latest version is
good. Here we use `grpcio>=1.60.0` means in future if another dependency breaks
the compatibility with `grpc==1.60.0`, there will be a chance to install a
higher version of `grpcio`.
--
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]
