Skip to content
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

Sync upstream repo #13

Merged
merged 16 commits into from
Feb 13, 2024
Merged

Sync upstream repo #13

merged 16 commits into from
Feb 13, 2024

Conversation

brijesh-vora-sp
Copy link

Syncing upstream repo. Saw that the fastapi was already updated in upstream: feast-dev#3938.

woop and others added 16 commits January 30, 2024 14:24
…t-dev#3896)

* [docs] updated development guide to include compiling the protos

Signed-off-by: franciscojavierarceo <[email protected]>

* adding note on spinning up the docker image

Signed-off-by: franciscojavierarceo <[email protected]>

---------

Signed-off-by: franciscojavierarceo <[email protected]>
chore: set upper bound for moto package

Signed-off-by: Hai Nguyen <[email protected]>
Modify the README to point to correct project list

Signed-off-by: Twinkll Sisodia <[email protected]>
@brijesh-vora-sp brijesh-vora-sp changed the title Brijesh.vora/saasmlops 1113 Sync upstream repo Feb 12, 2024
@dawid-laszuk-sp
Copy link

Can you please explain how you synced from the upstream? There's a lot of changes and would like to be extra careful.

@brijesh-vora-sp
Copy link
Author

Yeah, there were a lot of changes and that's why I created a PR to review and be careful. I created a branch brijesh.vora/SAASMLOPS-1119 in github and sync the changes in that branch and than compared with our master branch and created a PR to review. It touches one bytewax file sdk/python/feast/infra/materialization/contrib/bytewax/bytewax_materialization_engine.py to review.

@brijesh-vora-sp brijesh-vora-sp merged commit 9cbd4dc into master Feb 13, 2024
17 checks passed
@alex-vinnik-sp alex-vinnik-sp deleted the brijesh.vora/SAASMLOPS-1113 branch February 13, 2024 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.