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

feat: Adding list_validation_references for default and sql registry #3436

Merged
merged 5 commits into from
Jan 8, 2023

Conversation

amommendes
Copy link
Contributor

@amommendes amommendes commented Jan 7, 2023

Signed-off-by: Amom Mendes amommendes@hotmail.com

What this PR does / why we need it:
Adding list_validation_references method which is not implemented on default and sql registry
Which issue(s) this PR fixes:

Fixes #3408

Signed-off-by: Amom Mendes <amommendes@hotmail.com>
Signed-off-by: Amom Mendes <amommendes@hotmail.com>
Signed-off-by: Amom Mendes <amommendes@hotmail.com>
@amommendes amommendes changed the title Adding list_validation_references for default and sql registry chore: Adding list_validation_references for default and sql registry Jan 7, 2023
@amommendes amommendes changed the title chore: Adding list_validation_references for default and sql registry feat: Adding list_validation_references for default and sql registry Jan 7, 2023
@amommendes
Copy link
Contributor Author

amommendes commented Jan 7, 2023

@sfc-gh-madkins and @adchia, I'm not sure if we should raise an exception if we don't have any ValidationReference in the project... Considering that _list_objects from sql registry returns an empty list if there are no rows from some table, I think that is not a problem to return an empty list as well ... WDYT?

Signed-off-by: Amom Mendes <amommendes@hotmail.com>
Copy link
Collaborator

@adchia adchia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@feast-ci-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adchia, amommendes

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@amommendes
Copy link
Contributor Author

Hey @adchia, I didn't find commits not signed... Is the merge blocked by this?

@sfc-gh-madkins sfc-gh-madkins enabled auto-merge (squash) January 7, 2023 15:56
@feast-ci-bot feast-ci-bot merged commit 21dd253 into feast-dev:master Jan 8, 2023
kevjumba pushed a commit that referenced this pull request Jan 31, 2023
# [0.29.0](v0.28.0...v0.29.0) (2023-01-31)

### Bug Fixes

* Add check for bool type in addition to sample ([#3452](#3452)) ([1c7c491](1c7c491))
* Buggy SQL for postgres source ([#3424](#3424)) ([1ea100e](1ea100e))
* Ensure no duplicates in `fv.schema` ([#3460](#3460)) ([08ffa8d](08ffa8d))
* Fix delete sfv twice issue ([#3466](#3466)) ([dfd5eae](dfd5eae))
* Stream feature view UI shows transformation issue ([#3464](#3464)) ([1ef5137](1ef5137))
* Update registry.refresh to have a default arg ([#3450](#3450)) ([2f7c4ed](2f7c4ed))
* Updating the batch field so that you can query create and event date. ([#3411](#3411)) ([01ab462](01ab462)), closes [#3401](#3401)

### Features

* Add data source search ([#3449](#3449)) ([fbbb293](fbbb293))
* Adding list_validation_references for default and sql registry ([#3436](#3436)) ([21dd253](21dd253))
* Make UI accessible behind proxy ([#3428](#3428)) ([753d8db](753d8db))
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.

SQL Registry + caching is missing the function: list_validation_references(
3 participants