ArcticDB icon indicating copy to clipboard operation
ArcticDB copied to clipboard

maint: Support protobuf Python library v5

Open poodlewars opened this issue 11 months ago • 8 comments

Is your feature request related to a problem? Please describe. Protobuf has released version 5 but we only support version 3 and 4. PR #1421 has constrained protobuf version for now.

We should relax this constraint that we are imposing on user's envs.

Our proto handling is quite complex - we compile Python proto stubs suitable for both proto 3 and proto 4 and choose which a user will "see" at runtime based on their installed proto version.

The solution should consider both wheels and Conda builds.

poodlewars avatar Mar 14 '24 10:03 poodlewars

Currently ArcticDB's conda packages are built for:

protobuf 5 has not yet been published on conda-forge. https://github.com/conda-forge/libprotobuf-feedstock/pull/213 is the first PR to start this work.

We might need to define a build variant for protobuf 3, 4, and 5 once this last version is supported.

jjerphan avatar Mar 15 '24 10:03 jjerphan

Related https://github.com/grpc/grpc/issues/36142.

jjerphan avatar Apr 03 '24 08:04 jjerphan

Hi, any movement on this? Just curious if there's a timeline

rhoens avatar May 31 '24 20:05 rhoens

Likely addressable for conda-forge's builds once https://github.com/conda-forge/protobuf-feedstock/pull/216 is completed.

jjerphan avatar Jun 03 '24 11:06 jjerphan

Upgrade to protobuf 5 for pip wheel has been done in #1635. Waiting for https://github.com/conda-forge/protobuf-feedstock/pull/216 for protobuf 5 upgrade of conda

muhammadhamzasajjad avatar Jul 08 '24 08:07 muhammadhamzasajjad

FYI https://github.com/conda-forge/protobuf-feedstock/pull/215 can also be watched (it was started before the bot opened https://github.com/conda-forge/protobuf-feedstock/pull/216).

jjerphan avatar Jul 08 '24 08:07 jjerphan

Protobuf 5 is being released on conda-forge thanks to https://github.com/conda-forge/protobuf-feedstock/pull/215.

jjerphan avatar Sep 19 '24 12:09 jjerphan

ArcticDB can be build for libprotobuf 5 on the feedstock: https://github.com/conda-forge/arcticdb-feedstock/pull/298

jjerphan avatar Oct 01 '24 12:10 jjerphan

See: https://github.com/conda-forge/arcticdb-feedstock/pull/307

jjerphan avatar Oct 07 '24 09:10 jjerphan