◐ Shell reader mode source ↗

feat: Bump psycopg2 to psycopg3 for all Postgres components#4303

Merged

HaoXuAI merged 21 commits into

feast-dev:masterfrom

job-almekinders:psycopg2-3-ready

Jul 1, 2024

job-almekinders commented Jun 21, 2024

Copy link Copy Markdown

Contributor

What this PR does / why we need it:

This PR upgrades the psycopg2 dependency to the newer psycopg3 dependency. See here for more information on the differences between the two versions.

This is the 1st out of 2 PRs, required to enable async feature retrieval for the Postgres Online Store.

While here:

Makefile: Remove additional whitespace and exclude Snowflake tests for Postgres offline integration tests
Make some functions more readable, and add some additional docstrings and typehints.
Make the batch_size argument configurable for the postgres online store materialization function. This Fixes: Discussion: Pushing batches of data to online store: Should conn.commit() happen in the for loop or after? #4036

Additional remarks

The changes in this commit are related to the linter. In psycopg3, stricter type hints on the Cursor object require handling cases where cursor.description might be None. Although psycopg2 could also return None for this, it wasn't previously accounted for.

Which issue(s) this PR fixes:

1st out of 2 PRs required to fix #4260

job-almekinders commented Jun 21, 2024

View reviewed changes

job-almekinders left a comment

Copy link Copy Markdown

Contributor Author

Some additional clarifications from my side!

job-almekinders force-pushed the psycopg2-3-ready branch from e2d2188 to 38a376f Compare June 21, 2024 13:49

tokoko reviewed Jun 21, 2024

View reviewed changes

tokoko reviewed Jun 21, 2024

View reviewed changes

tokoko added the ok-to-test label Jun 21, 2024

franciscojavierarceo reviewed Jun 23, 2024

View reviewed changes

job-almekinders requested review from TomSteenbergen, franciscojavierarceo and tokoko June 25, 2024 09:59

tokoko reviewed Jun 25, 2024

View reviewed changes

job-almekinders mentioned this pull request Jun 25, 2024

feat: Add Postgres Online Store Async Feature Retrieval job-almekinders/feast#2

Closed

tokoko approved these changes Jun 25, 2024

View reviewed changes

tokoko left a comment

Copy link Copy Markdown

Collaborator

LGTM

job-almekinders commented Jun 26, 2024

Copy link Copy Markdown

Contributor Author

Hey @franciscojavierarceo, would you perhaps be able to do another pass on this PR? :)

HaoXuAI commented Jun 26, 2024

Copy link Copy Markdown

Collaborator

Looks like I merged the other pr caused conflicts here. mind fix it then we can merge it?

job-almekinders added 2 commits June 27, 2024 08:41


          Makefile: Formatting

…

9cdceab

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


          Makefile: Exclude Snowflake tests for postgres offline store tests

…

dca9c9a

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

job-almekinders force-pushed the psycopg2-3-ready branch from b94fb44 to 7165c51 Compare June 27, 2024 06:53

job-almekinders commented Jun 27, 2024

Copy link Copy Markdown

Contributor Author

Looks like I merged the other pr caused conflicts here. mind fix it then we can merge it?

I just pushed the update! @HaoXuAI

job-almekinders added 12 commits June 27, 2024 08:54


          Bootstrap: Use conninfo

…

fc65cfc

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


          Tests: Make connection string compatible with psycopg3

…

a3ea80d

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


          Tests: Test connection type pool and singleton

…

e53d9e6

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


          Global: Replace conn.set_session() calls to be psycopg3 compatible

…

59cbd10

Set connection read only

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


          Offline: Use psycopg3

…

0f86e9e

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


          Online: Use psycopg3

…

cd91fdc

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


          Online: Restructure online_write_batch

…

3504c77

Addition

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


          Online: Use correct placeholder

…

6e45f8e

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


          Online: Handle bytes properly in online_read()

…

c755fcd

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


          Online: Whitespace

…

acd4a8f

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


          Online: Open ConnectionPool

…

36147ef

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


          Online: Add typehint

…

d3fd7e7

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

25 hidden items Load more…

job-almekinders added 7 commits June 27, 2024 08:54


          Utils: Use psycopg3

…

0a9bced

Use new ConnectionPool

Pass kwargs as named argument

Use executemany over execute_values

Remove not-required open argument in psycopg.connect

Improve

Use SpooledTemporaryFile

Use max_size and add docstring

Properly write with StringIO

Utils: Use SpooledTemporaryFile over StringIO object

Add replace

Fix df_to_postgres_table

Remove import

Utils

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


          Lint: Raise exceptions if cursor returned no columns or rows

…

af136da

Add log statement

Lint: Fix _to_arrow_internal

Lint: Fix _get_entity_df_event_timestamp_range

Update exception

Use ZeroColumnQueryResult

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


          Add comment on +psycopg string

…

0926a15

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


          Docs: Remove mention of psycopg2

…

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


          Lint: Fix

…

915454c

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


          Default to postgresql+psycopg and log warning

…

d8e6619

Update warning

Fix

Format warning

Add typehints

Use better variable name

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


          Solve merge conflicts

…

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

job-almekinders force-pushed the psycopg2-3-ready branch from 7165c51 to 3328530 Compare June 27, 2024 06:54

job-almekinders commented Jun 27, 2024 •

edited

Loading

Copy link Copy Markdown

Contributor Author

We tried to install and run this feature branch already in one of our downstream projects. We noticed that there is an (edge) case where the required dependency combination of psycopg and sqlalchemy is not working properly. This is related to this discussion.

One of Feast' requirements is SQLAlchemy>=1. This enables feast to install either version 1 or 2. With this feature branch, the psycopg2 dependency is replaced with psycopg3. The path parameter of the RegistryConfig is being passed to sqlalchemy directly. However, the psycopg3 driver is only available for SQLAlchemy>=2.
If users in downstream projects have an additional dependency of SQLAlchemy<2, Feast is still able to be installed properly, however, with SQLAlchemy==1. This version doesn't support the psycop3 driver however. Since psycopg2 is no dependency of feast anymore, the user will not be able to connect to the registry out of the box.

The question now is how to handle this. We think there are a number of potential solutions:

Update the validator, add add additional version checks for sqlalchemy and/or psycopg2/3, and based on that decide which driver to use. For example:
a) If SQLAlchemy>=2, use psycopg3 (which is always installed with feast with postgres extras)
b) If SQLAlchemy<2, check for availability of psycopg2.
If psycopg2 is available, add this to the path prefix and continue.
If psycopg2 is not available, raise an Exception and instruct the user to add it to their own dependencies.
Users have to install the psycopg2 dependency themselves in their downstream project, and use the postgresql+psycopg2 prefix for the path parameter of the RegistryConfig. Potentially, we could also decide to drop the added validator, and instead communicate to the user how to handle this.

There are more ways to tackle this issue, but I'm curious to hear your thoughts on what you think might be the best way forward.

@tokoko I would love to hear your thoughts as well on this matter if you have the time :)

job-almekinders mentioned this pull request Jun 27, 2024

feat: Bump psycopg2 to psycopg3 for all Postgres components job-almekinders/feast#1

Closed

HaoXuAI commented Jun 27, 2024

Copy link Copy Markdown

Collaborator

We tried to install and run this feature branch already in one of our downstream projects. We noticed that there is an (edge) case where the required dependency combination of psycopg and sqlalchemy is not working properly. This is related to this discussion.

One of Feast' requirements is SQLAlchemy>=1. This enables feast to install either version 1 or 2. With this feature branch, the psycopg2 dependency is replaced with psycopg3. The path parameter of the RegistryConfig is being passed to sqlalchemy directly. However, the psycopg3 driver is only available for SQLAlchemy>=2. If users in downstream projects have an additional dependency of SQLAlchemy<2, Feast is still able to be installed properly, however, with SQLAlchemy==1. This version doesn't support the psycop3 driver however. Since psycopg2 is no dependency of feast anymore, the user will not be able to connect to the registry out of the box.

The question now is how to handle this. We think there are a number of potential solutions:

Update the validator, add add additional version checks for sqlalchemy and/or psycopg2/3, and based on that decide which driver to use. For example:
a) If SQLAlchemy>=2, use psycopg3 (which is always installed with feast with postgres extras)
b) If SQLAlchemy<2, check for availability of psycopg2.
If psycopg2 is available, add this to the path prefix and continue.
If psycopg2 is not available, raise an Exception and instruct the user to add it to their own dependencies.

Users have to install the psycopg2 dependency themselves in their downstream project, and use the postgresql+psycopg2 prefix for the path parameter of the RegistryConfig. Potentially, we could also decide to drop the added validator, and instead communicate to the user how to handle this.

There are more ways to tackle this issue, but I'm curious to hear your thoughts on what you think might be the best way forward.

@tokoko I would love to hear your thoughts as well on this matter if you have the time :)

any reason we are not able to bump sqlAlchemy to 2.0+?

tokoko commented Jun 27, 2024

Copy link Copy Markdown

Collaborator

@job-almekinders Thanks for pointing that out. However, I don't believe this warrants any action from us. Those kinds of diamond dependencies are just part of life in libraries like this, unfortunately. I'm sure there are a few others similar to this lurking around that we haven't noticed before.

If a user has some hard dependency on sqlalchemy1 and also uses postgres in feast only for the registry, they are free to forgo postgres extra, install psycopg2 instead and set up sql registry path accordingly.

any reason we are not able to bump sqlAlchemy to 2.0+?

I guess we can, but as long as we don't absolutely need to, it's better to leave it as is not to cause unnecessary diamond dependency problems to downstream libraries.

TomSteenbergen commented Jun 27, 2024

Copy link Copy Markdown

Contributor

any reason we are not able to bump sqlAlchemy to 2.0+?

@HaoXuAI This will probably break the Snowflake integration, as SQLAlchemy 2 isn't supported yet by the Snowflake Python libraries. See this relevant issue: snowflakedb/snowflake-sqlalchemy#380

job-almekinders commented Jun 27, 2024

Copy link Copy Markdown

Contributor Author

@job-almekinders Thanks for pointing that out. However, I don't believe this warrants any action from us. Those kinds of diamond dependencies are just part of life in libraries like this, unfortunately. I'm sure there are a few others similar to this lurking around that we haven't noticed before.

If a user has some hard dependency on sqlalchemy1 and also uses postgres in feast only for the registry, they are free to forgo postgres extra, install psycopg2 instead and set up sql registry path accordingly.

That makes sense !

If this is no blocker then I think we are good to move forward with this PR, at least from our end :)

HaoXuAI commented Jun 27, 2024

Copy link Copy Markdown

Collaborator

any reason we are not able to bump sqlAlchemy to 2.0+?

@HaoXuAI This will probably break the Snowflake integration, as SQLAlchemy 2 isn't supported yet by the Snowflake Python libraries. See this relevant issue: snowflakedb/snowflake-sqlalchemy#380

I think snowflake can use snowflake-python module which doesn't depend on sqlAchemy.
Also since we are on python 3.9+ that shouldn't be many cases which still depending on sqlAchemy 1.4?

tokoko commented Jul 1, 2024

Copy link Copy Markdown

Collaborator

I think snowflake can use snowflake-python module which doesn't depend on sqlAchemy.

I think he meant snowflake-backed sql registry, not a separate snowflake registry.

Also since we are on python 3.9+ that shouldn't be many cases which still depending on sqlAchemy 1.4?

I'm fine with removing EOL versions if that will get us anything. Probably best to follow up in a different issue, though.

@HaoXuAI This is good to merge, right?

HaoXuAI approved these changes Jul 1, 2024

View reviewed changes

HaoXuAI merged commit 9451d9c into feast-dev:master Jul 1, 2024

tsisodia10 pushed a commit to tsisodia10/feast that referenced this pull request Jul 23, 2024


          feat: Bump psycopg2 to psycopg3 for all Postgres components (feast-de…

…

de203dc

…v#4303)

* Makefile: Formatting

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

* Makefile: Exclude Snowflake tests for postgres offline store tests

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

* Bootstrap: Use conninfo

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

* Tests: Make connection string compatible with psycopg3

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

* Tests: Test connection type pool and singleton

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

* Global: Replace conn.set_session() calls to be psycopg3 compatible

Set connection read only

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

* Offline: Use psycopg3

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

* Online: Use psycopg3

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

* Online: Restructure online_write_batch

Addition

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

* Online: Use correct placeholder

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

* Online: Handle bytes properly in online_read()

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

* Online: Whitespace

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

* Online: Open ConnectionPool

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

* Online: Add typehint

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

* Utils: Use psycopg3

Use new ConnectionPool

Pass kwargs as named argument

Use executemany over execute_values

Remove not-required open argument in psycopg.connect

Improve

Use SpooledTemporaryFile

Use max_size and add docstring

Properly write with StringIO

Utils: Use SpooledTemporaryFile over StringIO object

Add replace

Fix df_to_postgres_table

Remove import

Utils

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

* Lint: Raise exceptions if cursor returned no columns or rows

Add log statement

Lint: Fix _to_arrow_internal

Lint: Fix _get_entity_df_event_timestamp_range

Update exception

Use ZeroColumnQueryResult

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

* Add comment on +psycopg string

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

* Docs: Remove mention of psycopg2

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

* Lint: Fix

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

* Default to postgresql+psycopg and log warning

Update warning

Fix

Format warning

Add typehints

Use better variable name

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

* Solve merge conflicts

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

---------

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

tsisodia10 pushed a commit to tsisodia10/feast that referenced this pull request Jul 23, 2024


          feat: Bump psycopg2 to psycopg3 for all Postgres components (feast-de…

…

6c94cdf

…v#4303)

* Makefile: Formatting

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

* Makefile: Exclude Snowflake tests for postgres offline store tests

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

* Bootstrap: Use conninfo

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

* Tests: Make connection string compatible with psycopg3

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

* Tests: Test connection type pool and singleton

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

* Global: Replace conn.set_session() calls to be psycopg3 compatible

Set connection read only

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

* Offline: Use psycopg3

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

* Online: Use psycopg3

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

* Online: Restructure online_write_batch

Addition

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

* Online: Use correct placeholder

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

* Online: Handle bytes properly in online_read()

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

* Online: Whitespace

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

* Online: Open ConnectionPool

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

* Online: Add typehint

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

* Utils: Use psycopg3

Use new ConnectionPool

Pass kwargs as named argument

Use executemany over execute_values

Remove not-required open argument in psycopg.connect

Improve

Use SpooledTemporaryFile

Use max_size and add docstring

Properly write with StringIO

Utils: Use SpooledTemporaryFile over StringIO object

Add replace

Fix df_to_postgres_table

Remove import

Utils

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

* Lint: Raise exceptions if cursor returned no columns or rows

Add log statement

Lint: Fix _to_arrow_internal

Lint: Fix _get_entity_df_event_timestamp_range

Update exception

Use ZeroColumnQueryResult

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

* Add comment on +psycopg string

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

* Docs: Remove mention of psycopg2

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

* Lint: Fix

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

* Default to postgresql+psycopg and log warning

Update warning

Fix

Format warning

Add typehints

Use better variable name

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

* Solve merge conflicts

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

---------

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

franciscojavierarceo pushed a commit that referenced this pull request Jul 31, 2024


          chore(release): release 0.40.0

…

32ec0e0

# [0.40.0](v0.39.0...v0.40.0) (2024-07-31)

### Bug Fixes

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

### Features

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

### Reverts

* Revert "fix: Avoid XSS attack from Jinjin2's Environment()." ([#4357](#4357)) ([cdeab48](cdeab48)), closes [#4355](#4355)

redhatHameed pushed a commit to RHEcosystemAppEng/feast that referenced this pull request Aug 5, 2024


          chore(release): release 0.40.0

…

cd6bbdd

# [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)

shuchu pushed a commit to shuchu/feast that referenced this pull request Aug 14, 2024


          chore(release): release 0.40.0

…

cae788f

# [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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

tokoko tokoko approved these changes

franciscojavierarceo Awaiting requested review from franciscojavierarceo

Assignees

No one assigned

Labels

Projects

Milestone

No milestone