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

Updated versions of dependencies #34

Merged
merged 1 commit into from
Jan 3, 2023

Conversation

EvanHStanton
Copy link
Contributor

@EvanHStanton EvanHStanton commented Jan 3, 2023

  • Updated dependencies.

@ChuckHend: I was interested in this particular Cargo.toml file due to its use of the object relational mapper crate, diesel. If you have not already come across the sea-orm crate, it might be interesting to take a quick peek. It is an alternative that is built from the ground up with async in mind, and built upon sqlx, which the sea-orm docs notes works well with actix, async-std, and tokio.

https://www.sea-ql.org/SeaORM/

diesel might also check these boxes, and it should be noted that there is an extension called diesel-async, which may address well one of the critiques, i.e., that it wasn't originally built with async in mind.

Copy link
Member

@ChuckHend ChuckHend left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks for pointer to sea-orm and updating the deps!

Copy link
Member

@ianstanton ianstanton left a comment

Choose a reason for hiding this comment

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

💥

@ianstanton ianstanton merged commit 5dc41ff into tembo-io:main Jan 3, 2023
sjmiller609 pushed a commit that referenced this pull request Dec 5, 2023
* update extension object

* uncomment stuff

* update test

* update spec

* build

* update the version in tests

* Update reconciler-test.yml

* roll back comment

* Update reconciler-test.yml

* Update coredb_crd.rs

---------

Co-authored-by: Ian Stanton <ian@coredb.io>
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.

3 participants