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: Merged changes from feast open source #139

Merged
merged 121 commits into from
Sep 27, 2024
Merged

Conversation

EXPEbdodla
Copy link
Collaborator

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes

tokoko and others added 30 commits July 18, 2024 14:46
port mssql offline store to ibis

Signed-off-by: tokoko <togurg14@freeuni.edu.ge>
remove firebase-admin from ci dependencies

Signed-off-by: tokoko <togurg14@freeuni.edu.ge>
…v#4362)

Bumps [ws](https://github.com/websockets/ws) from 7.5.7 to 7.5.10.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.5.7...7.5.10)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
increment operator to v0.39.0

Signed-off-by: Tommy Hughes <tohughes@redhat.com>
# [0.40.0](feast-dev/feast@v0.39.0...v0.40.0) (2024-07-31)

### Bug Fixes

* Added missing type ([feast-dev#4315](feast-dev#4315)) ([86af60a](feast-dev@86af60a))
* Avoid XSS attack from Jinjin2's Environment(). ([feast-dev#4355](feast-dev#4355)) ([40270e7](feast-dev@40270e7))
* CGO Memory leak issue in GO Feature server ([feast-dev#4291](feast-dev#4291)) ([43e198f](feast-dev@43e198f))
* Deprecated the datetime.utcfromtimestamp(). ([feast-dev#4306](feast-dev#4306)) ([21deec8](feast-dev@21deec8))
* Fix SQLite import issue ([feast-dev#4294](feast-dev#4294)) ([398ea3b](feast-dev@398ea3b))
* Increment operator to v0.39.0 ([feast-dev#4368](feast-dev#4368)) ([3ddb4fb](feast-dev@3ddb4fb))
* Minor typo in the unit test. ([feast-dev#4296](feast-dev#4296)) ([6c75e84](feast-dev@6c75e84))
* OnDemandFeatureView type inference for array types ([feast-dev#4310](feast-dev#4310)) ([c45ff72](feast-dev@c45ff72))
* Remove redundant batching in PostgreSQLOnlineStore.online_write_batch and fix progress bar ([feast-dev#4331](feast-dev#4331)) ([0d89d15](feast-dev@0d89d15))
* Remove typo. ([feast-dev#4351](feast-dev#4351)) ([92d17de](feast-dev@92d17de))
* Retire the datetime.utcnow(). ([feast-dev#4352](feast-dev#4352)) ([a8bc696](feast-dev@a8bc696))
* Update dask version to support pandas 1.x ([feast-dev#4326](feast-dev#4326)) ([a639d61](feast-dev@a639d61))
* Update Feast object metadata in the registry ([feast-dev#4257](feast-dev#4257)) ([8028ae0](feast-dev@8028ae0))
* Using one single function call for utcnow(). ([feast-dev#4307](feast-dev#4307)) ([98ff63c](feast-dev@98ff63c))

### Features

* Add async feature retrieval for Postgres Online Store ([feast-dev#4327](feast-dev#4327)) ([cea52e9](feast-dev@cea52e9))
* Add Async refresh to Sql Registry ([feast-dev#4251](feast-dev#4251)) ([f569786](feast-dev@f569786))
* Add SingleStore as an OnlineStore ([feast-dev#4285](feast-dev#4285)) ([2c38946](feast-dev@2c38946))
* Add Tornike to maintainers.md ([feast-dev#4339](feast-dev#4339)) ([8e8c1f2](feast-dev@8e8c1f2))
* Bump psycopg2 to psycopg3 for all Postgres components ([feast-dev#4303](feast-dev#4303)) ([9451d9c](feast-dev@9451d9c))
* Entity key deserialization ([feast-dev#4284](feast-dev#4284)) ([83fad15](feast-dev@83fad15))
* Ignore paths feast apply ([feast-dev#4276](feast-dev#4276)) ([b4d54af](feast-dev@b4d54af))
* Move get_online_features to OnlineStore interface ([feast-dev#4319](feast-dev#4319)) ([7072fd0](feast-dev@7072fd0))
* Port mssql contrib offline store to ibis ([feast-dev#4360](feast-dev#4360)) ([7914cbd](feast-dev@7914cbd))

### Reverts

* Revert "fix: Avoid XSS attack from Jinjin2's Environment()." ([feast-dev#4357](feast-dev#4357)) ([cdeab48](feast-dev@cdeab48)), closes [feast-dev#4355](feast-dev#4355)
* fix: revert java testcontainers to use docker-based compose

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: update testcontainers-java to Compose v2

Signed-off-by: tokoko <togurgenidze@gmail.com>

---------

Signed-off-by: tokoko <togurgenidze@gmail.com>
Co-authored-by: tokoko <togurgenidze@gmail.com>
…st-dev#4378)

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>
)

Signed-off-by: Tommy Hughes <tohughes@redhat.com>
…4366)

feat: instrument feature store

This commit adds opentelemetry to monitor Feast

Signed-off-by: Twinkll Sisodia <tsisodia@redhat.com>
* docs: Adding some content about the push vs pull model

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* feat: Updating SQLite-Vec to stable release version:

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing push model changes from other branch

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
* docs: Adding some content about the push vs pull model

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* checking in progress...all of this was mostly autogenerated

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update push-vs-pull-model.md

* Update push-vs-pull-model.md

* Update push-vs-pull-model.md

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
… annotation (feast-dev#4391)

fix: Using get_type_hints instead of inspect for udf return type

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>
* Apply fix

Signed-off-by: Job Almekinders <job.almekinders@teampicnic.com>

* Add special characters to postgres online store test

Signed-off-by: Job Almekinders <job.almekinders@teampicnic.com>

* Fix linting error

Signed-off-by: Job Almekinders <job.almekinders@teampicnic.com>

---------

Signed-off-by: Job Almekinders <job.almekinders@teampicnic.com>
…thentication (feast-dev#4384)

* allow feast snowflake to read in byte string for private-key authentication

Signed-off-by: Artur <artur.kolakowski@medely.com>

* Update type hint for  to use Union instead of | syntax

Signed-off-by: Artur <artur.kolakowski@medely.com>

* Update type hint for private_key to use Union instead of | syntax

Signed-off-by: Artur <artur.kolakowski@medely.com>

* Update type hint in parse_private_key_path

Signed-off-by: Artur <artur.kolakowski@medely.com>

* added private_key_content in Snowflake configs to support key-pair auth by reading in byte string

Signed-off-by: Artur <artur.kolakowski@medely.com>

* fix incompatible linting types

Signed-off-by: Artur <artur.kolakowski@medely.com>

* remove unused Union import

Signed-off-by: Artur <artur.kolakowski@medely.com>

* fix formating

Signed-off-by: Artur <artur.kolakowski@medely.com>

---------

Signed-off-by: Artur <artur.kolakowski@medely.com>
Co-authored-by: Artur <artur.kolakowski@medely.com>
…erns and feature transformations (feast-dev#4400)

* merging changes

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* merging

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Adding some clarity.

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Update feature-transformetion.md (feast-dev#4405)

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated readme

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated summary and readme

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated docs

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Updated readme

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated more

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated transformation

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated urls

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* refactoring and renaming architecture-and-components to components and architecture

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated urls

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* moving files from architecture-and-components to components/

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* had a typo in components

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Cleaned everything up

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
* Create ADOPTERS.md

* Update ADOPTERS.md
…re-yaml overrides (feast-dev#4413)

* fix: using repo_config parameter in teardown to allow for feature-store-yaml overrides

Signed-off-by: Dan Baron <dan.baron@starlingbank.com>

* fix: fixing linting and formatting issues in tests

Signed-off-by: Dan Baron <dan.baron@starlingbank.com>

* fix: removing unnecessary Path object construction

Signed-off-by: Dan Baron <dan.baron@starlingbank.com>

---------

Signed-off-by: Dan Baron <dan.baron@starlingbank.com>
…4416)

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
* fix: Remove pytz.

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>

* fix: Keep the pytz.UTC part in dask.py

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>

---------

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>
Bhargav Dodla and others added 26 commits September 17, 2024 16:50
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
* fix: Removed protobuf as a required dependency

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Removed install-protoc-dependencies target

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
…ssion flow (feast-dev#4531)

* Logger setting for feature servers

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added/updated logger for permission flow

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* set defualt logger level for feature servers as WARNING

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

---------

Signed-off-by: Abdul Hameed <ahameed@redhat.com>
…tations (feast-dev#4537)

* fix: Removed protobuf as a required dependency

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Removed install-protoc-dependencies target

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Ran lock python dependencies to correct dependencies

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Adding protobuf<5 as a required dependency due to snowflake limitations

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
…dev#4538)

minor: Added indexes to sql tables to optimize query execution

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>
…ev#4536)

* Add docs example of how to use tags with feature views

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Add docs example of how to use tags with feature views
- changes following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

---------

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
…dev#4544)

* fix: Fix for SQL registry initialization fails feast-dev#4543

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Removed combined_sql_fixtures

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Added protobuf dependency to pyproject.toml

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
@EXPEbdodla EXPEbdodla merged commit 2a5cfed into master Sep 27, 2024
23 checks passed
@EXPEbdodla EXPEbdodla deleted the feast_merge_sep_2024 branch September 27, 2024 16:17
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.