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

fix: Feast merge jun 24, 2024 #115

Merged
merged 236 commits into from
Jul 18, 2024
Merged

fix: Feast merge jun 24, 2024 #115

merged 236 commits into from
Jul 18, 2024

Conversation

EXPEbdodla
Copy link
Collaborator

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

franciscojavierarceo and others added 30 commits January 31, 2024 16:23
…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]>
…ev#3936)

rewrite spark materilization engine to use mapInPandas

Signed-off-by: tokoko <[email protected]>
…#3922)

* chore: Loosen fsspec requirements to allow recent releases

(I'm not sure the project has a super consistent pattern for when to
specify a maximum version, but was going for the smallest possible
change.)

Signed-off-by: Chris Burroughs <[email protected]>

* drop redundant fsspec now that this is in another extra

Signed-off-by: Chris Burroughs <[email protected]>

* post rebase regen

Signed-off-by: Chris Burroughs <[email protected]>

---------

Signed-off-by: Chris Burroughs <[email protected]>
…ev#3954)

* fix: Fix typo as the cli does not support shortcut-f option.

Signed-off-by: Shuchu Han <[email protected]>

* fix: add -f option as a shortcut of feature-store-yaml.

Signed-off-by: Shuchu Han <[email protected]>

---------

Signed-off-by: Shuchu Han <[email protected]>
* fetch_arrow_all returns empty table

Signed-off-by: Chester Ong <[email protected]>

* fix spark_kafka_processor typing errors

Signed-off-by: Chester Ong <[email protected]>

* fix correct return type

Signed-off-by: Chester Ong <[email protected]>

* revert _to_arrow_internal

Signed-off-by: Chester Ong <[email protected]>

* revert kafkaStreamProcessor changes, change base type instead

Signed-off-by: Chester Ong <[email protected]>

---------

Signed-off-by: Chester Ong <[email protected]>
…ev#3953)

* using version args to install the correct feast version

Signed-off-by: Chester Ong <[email protected]>

* revert the COPY command

Signed-off-by: Chester Ong <[email protected]>

---------

Signed-off-by: Chester Ong <[email protected]>
…pe features. (feast-dev#3964)

Bugfix for grabbing historical data from Snowflake with array type features that are null for an entity.



Update docs to reflect array support in Snowflake

Signed-off-by: john.lemmon <[email protected]>
* decouple transformation from odfvs

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

* OnDemandFeatureView: keep udf and udf_string parameters for backwards compatibility

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

* fix linting issues

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

* remove unused import in registry protos

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

---------

Signed-off-by: tokoko <[email protected]>
…east-dev#3968)

Revert "feat: Updating dependencies so that feast can be run on 3.11. (feast-dev#3958)"

This reverts commit 59639db.

Signed-off-by: franciscojavierarceo <[email protected]>
fix: quickstart notebook example
Removed unnecessary comma in notebook

Signed-off-by: Theodor Mihalache <[email protected]>
EXPEbdodla and others added 24 commits June 13, 2024 06:28
…ev#4274)

* fix: Handles null values in data during GO Feature retrieval

Signed-off-by: Bhargav Dodla <[email protected]>

* fix: Fixed formatting issues

Signed-off-by: Bhargav Dodla <[email protected]>

* fix: Fixed linting issues

Signed-off-by: Bhargav Dodla <[email protected]>

---------

Signed-off-by: Bhargav Dodla <[email protected]>
Co-authored-by: Bhargav Dodla <[email protected]>
* update gitpod.yaml with uv and pixi

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

* fix linter issues

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

* change precommit commands

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

---------

Signed-off-by: tokoko <[email protected]>
…erving (feast-dev#4268)

chore: Bump com.azure:azure-identity in /java/serving

Bumps [com.azure:azure-identity](https://github.com/Azure/azure-sdk-for-java) from 1.11.3 to 1.12.2.
- [Release notes](https://github.com/Azure/azure-sdk-for-java/releases)
- [Commits](Azure/azure-sdk-for-java@azure-identity_1.11.3...azure-identity_1.12.2)

---
updated-dependencies:
- dependency-name: com.azure:azure-identity
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: Added offline store remote deployment functionly using arrow flight server and client

Signed-off-by: Abdul Hameed <[email protected]>

* Initial functional commit for remote get_historical_features

Signed-off-by: Abdul Hameed <[email protected]>

* remote offline store example

Signed-off-by: Abdul Hameed <[email protected]>

* removing unneeded test code and fixinf impotrts

Signed-off-by: Abdul Hameed <[email protected]>

* call do_put only once, postpone the invocation of do_put and simplified _make_flight_info

Signed-off-by: Abdul Hameed <[email protected]>

* added primitive parameters to the command descriptor

Signed-off-by: Abdul Hameed <[email protected]>

* removed redundant param

Signed-off-by: Abdul Hameed <[email protected]>

* Initial skeleton of unit test for offline server

Signed-off-by: Abdul Hameed <[email protected]>

* added unit test for offline store remote client

Signed-off-by: Abdul Hameed <[email protected]>

* testing all offlinestore APIs

Signed-off-by: Abdul Hameed <[email protected]>

* integrated comments

Signed-off-by: Abdul Hameed <[email protected]>

* Updated remote offline server readme with the capability to init with an environment variable

Signed-off-by: Theodor Mihalache <[email protected]>
Signed-off-by: Abdul Hameed <[email protected]>

* added    RemoteOfflineStoreDataSourceCreator,
use feature_view_names to transfer feature views and remove dummies

Signed-off-by: Abdul Hameed <[email protected]>

* added missing CI requirement

Signed-off-by: Abdul Hameed <[email protected]>

* fixed linter

Signed-off-by: Abdul Hameed <[email protected]>

* fixed multiprocess CI requirement

Signed-off-by: Abdul Hameed <[email protected]>

* feat: Added offline store remote deployment functionly using arrow flight server and client

Signed-off-by: Abdul Hameed <[email protected]>

* fix test errors

Signed-off-by: Abdul Hameed <[email protected]>

* managing feature view aliases and restored skipped tests

Signed-off-by: Abdul Hameed <[email protected]>

* fixced linter issue

Signed-off-by: Abdul Hameed <[email protected]>

* fixed broken test

Signed-off-by: Abdul Hameed <[email protected]>

* added supported deployment modes using helm chart for  online (default), offline, ui and registry

Signed-off-by: Abdul Hameed <[email protected]>

* updated the document for offline remote server

Signed-off-by: Abdul Hameed <[email protected]>

* added the document for remote offline server

Signed-off-by: Abdul Hameed <[email protected]>

* rebase and fix conflicts

Signed-off-by: Abdul Hameed <[email protected]>

* feat: Added offline store remote deployment functionly using arrow flight server and client

Signed-off-by: Abdul Hameed <[email protected]>

* added unit test for offline store remote client

Signed-off-by: Abdul Hameed <[email protected]>

* added    RemoteOfflineStoreDataSourceCreator,
use feature_view_names to transfer feature views and remove dummies

Signed-off-by: Abdul Hameed <[email protected]>

* feat: Added offline store remote deployment functionly using arrow flight server and client

Signed-off-by: Abdul Hameed <[email protected]>

* Added missing remote offline store apis implementation

Signed-off-by: Theodor Mihalache <[email protected]>
Signed-off-by: Abdul Hameed <[email protected]>

* Fixed tests

Signed-off-by: Theodor Mihalache <[email protected]>
Signed-off-by: Abdul Hameed <[email protected]>

* Implemented PR change proposal

Signed-off-by: Theodor Mihalache <[email protected]>
Signed-off-by: Abdul Hameed <[email protected]>

* Implemented PR change proposal

Signed-off-by: Theodor Mihalache <[email protected]>
Signed-off-by: Abdul Hameed <[email protected]>

* updated example readme file

Signed-off-by: Abdul Hameed <[email protected]>

* Implemented PR change proposal

Signed-off-by: Theodor Mihalache <[email protected]>
Signed-off-by: Abdul Hameed <[email protected]>

* fixing the integration tests

Signed-off-by: Abdul Hameed <[email protected]>

* Fixed OfflineServer teardown

Signed-off-by: Theodor Mihalache <[email protected]>

* updated the document for remote offline feature server and client

Signed-off-by: Abdul Hameed <[email protected]>

* Implemented PR change proposal

Signed-off-by: Theodor Mihalache <[email protected]>

---------

Signed-off-by: Abdul Hameed <[email protected]>
Signed-off-by: Theodor Mihalache <[email protected]>
Co-authored-by: Daniele Martinoli <[email protected]>
Co-authored-by: Theodor Mihalache <[email protected]>
Co-authored-by: Theodor Mihalache <[email protected]>
* docs: Adding language recommendation

Signed-off-by: Francisco Javier Arceo <[email protected]>

* Update language.md

* Update docs/getting-started/architecture-and-components/overview.md

* Update docs/getting-started/faq.md

* Update language.md

* Update docs/getting-started/architecture-and-components/overview.md

---------

Signed-off-by: Francisco Javier Arceo <[email protected]>
* chore: Removing logging from feature server

Signed-off-by: Francisco Javier Arceo <[email protected]>

* linter

Signed-off-by: Francisco Javier Arceo <[email protected]>

---------

Signed-off-by: Francisco Javier Arceo <[email protected]>
Co-authored-by: Francisco Javier Arceo <[email protected]>
add tags filtering capability to 'list' for objects

Signed-off-by: Tommy Hughes <[email protected]>
# [0.39.0](feast-dev/feast@v0.38.0...v0.39.0) (2024-06-18)

### Bug Fixes

* Feast UI importlib change ([feast-dev#4248](feast-dev#4248)) ([5d486b8](feast-dev@5d486b8))
* Feature server no_feature_log argument error ([feast-dev#4255](feast-dev#4255)) ([15524ce](feast-dev@15524ce))
* Feature UI Server image won't start in an OpenShift cluster ([feast-dev#4250](feast-dev#4250)) ([4891f76](feast-dev@4891f76))
* Handles null values in data during GO Feature retrieval ([feast-dev#4274](feast-dev#4274)) ([c491e57](feast-dev@c491e57))
* Make Java gRPC client use timeouts as expected ([feast-dev#4237](feast-dev#4237)) ([f5a37c1](feast-dev@f5a37c1))
* Remove self assignment code line. ([feast-dev#4238](feast-dev#4238)) ([e514f66](feast-dev@e514f66))
* Set default values for feature_store.serve() function ([feast-dev#4225](feast-dev#4225)) ([fa74438](feast-dev@fa74438))

### Features

* Add online_read_async for dynamodb ([feast-dev#4244](feast-dev#4244)) ([b5ef384](feast-dev@b5ef384))
* Add the ability to list objects by `tags` ([feast-dev#4246](feast-dev#4246)) ([fbf92da](feast-dev@fbf92da))
* Added deadline to gRPC Java client ([feast-dev#4217](feast-dev#4217)) ([ff429c9](feast-dev@ff429c9))
* Adding vector search for sqlite ([feast-dev#4176](feast-dev#4176)) ([2478831](feast-dev@2478831))
* Change get_online_features signature, move online retrieval functions to utils ([feast-dev#4278](feast-dev#4278)) ([7287662](feast-dev@7287662))
* Feature/adding remote online store ([feast-dev#4226](feast-dev#4226)) ([9454d7c](feast-dev@9454d7c))
* List all feature views ([feast-dev#4256](feast-dev#4256)) ([36a574d](feast-dev@36a574d))
* Make RegistryServer writable ([feast-dev#4231](feast-dev#4231)) ([79e1143](feast-dev@79e1143))
* Remote offline Store  ([feast-dev#4262](feast-dev#4262)) ([28a3d24](feast-dev@28a3d24))
* Set optional full-scan for deletion ([feast-dev#4189](feast-dev#4189)) ([b9cadd5](feast-dev@b9cadd5))
* Add new version of serialization and desrialization

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

* Add new version of serialization and desrialization

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

* fix test

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

* fix test

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

* add test

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

* add test

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

* update doc

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

---------

Signed-off-by: cmuhao <[email protected]>
Add parameters validation to OfflineServer

Signed-off-by: Theodor Mihalache <[email protected]>
@EXPEbdodla EXPEbdodla force-pushed the feast_merge_apr_2024 branch from 478eb64 to 59fa72a Compare June 27, 2024 02:47
@EXPEbdodla EXPEbdodla changed the title fix: Feast merge apr 2024 fix: Feast merge jun 24, 2024 Jun 27, 2024
@EXPEbdodla EXPEbdodla merged commit 498119c into master Jul 18, 2024
21 of 22 checks passed
@EXPEbdodla EXPEbdodla deleted the feast_merge_apr_2024 branch July 18, 2024 18:48
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.