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

Added an example for using PostgreSQL as a database registry for Feast #5

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

redhatHameed
Copy link
Contributor

@redhatHameed redhatHameed commented Feb 2, 2024

Added an example for using PostgreSQL as a database registry for Feast

@redhatHameed
Copy link
Contributor Author

@etirelli added this example to support SQL based registry

Copy link
Contributor

@jeremyary jeremyary left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Will leave to see if others want to review, then let it merge up later today. I couldn't find where materialize.png was referenced, so that may be extra but wanted to mention in case you had something in mind for it.

@jeremyary
Copy link
Contributor

jeremyary commented Feb 7, 2024

quick FYI - I'm disabling our plan/apply GH actions for now. They serve as good examples, but I don't think we have reason to live-sync our feature repo(s). LMK if you'd disagree, thanks!

@redhatHameed
Copy link
Contributor Author

LGTM, thanks! Will leave to see if others want to review, then let it merge up later today. I couldn't find where materialize.png was referenced, so that may be extra but wanted to mention in case you had something in mind for it.
Thanks for review.
Removed as this was extra added.

@redhatHameed
Copy link
Contributor Author

quick FYI - I'm disabling our plan/apply GH actions for now. They serve as good examples, but I don't think we have reason to live-sync our feature repo(s). LMK if you'd disagree, thanks!

Totally agree as like this example we are not using AWS or other cloud provider. For the time good to remove.

@jeremyary jeremyary merged commit f09c367 into RHEcosystemAppEng:main Feb 13, 2024
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.

2 participants