Skip to content

fix: Upperbound limit for protobuf generation #5309

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

HaoXuAI
Copy link
Collaborator

@HaoXuAI HaoXuAI commented Apr 30, 2025

What this PR does / why we need it:

Which issue(s) this PR fixes:

Misc

@HaoXuAI HaoXuAI changed the title Patch: Upperbound limit for protobuf generation fix: Upperbound limit for protobuf generation Apr 30, 2025
@ntkathole
Copy link
Contributor

Need to modify in https://github.com/feast-dev/feast/blob/master/pyproject.toml#L126
and probably also

feast/setup.py

Line 139 in a714d3a

GRPCIO_REQUIRED = [

@tchughesiv
Copy link
Contributor

tchughesiv commented Apr 30, 2025

@HaoXuAI fyi #5311
rebase

@HaoXuAI HaoXuAI requested a review from a team as a code owner May 1, 2025 06:09
@HaoXuAI HaoXuAI force-pushed the patch-grpcio-upper-bound branch from f54d672 to 30a50a4 Compare May 1, 2025 06:15
HaoXuAI added 3 commits April 30, 2025 23:19
Signed-off-by: HaoXuAI <[email protected]>
Signed-off-by: HaoXuAI <[email protected]>
Signed-off-by: HaoXuAI <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants