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

Rust / Rocket / Diesel / Mysql - Backend Application #272

Open
navinkumarr opened this Issue Sep 23, 2018 · 6 comments

Comments

Projects
None yet
4 participants
@anishkny

This comment has been minimized.

Member

anishkny commented Sep 23, 2018

Could you clarify how this is different from: https://github.com/gothinkster/rust-realworld-example-app

@navinkumarr

This comment has been minimized.

navinkumarr commented Sep 24, 2018

Here are a few points because of which I started working on this application. Don't see these in the example application you have posted above.

  1. It is built according to the Rocket framework standard (Trying to make a complete application with proper error handling, database connection pooling, support for master/slave architecture for database etc).
  2. I have seen many example applications using Postgres. This one has the implementation for Mysql (Even though using an ORM, a straightforward example still helps).
  3. Clean architecture (Could use this app architecture for writing bigger applications since it is properly layered).

Do let me know what you think @anishkny

@anishkny

This comment has been minimized.

Member

anishkny commented Sep 24, 2018

I am not an expert on Rust. It might be worth discussing with their contributors if this is different enough? @davidpodhola, @leia,

cc: @Cameron-C-Chapman

@davidpodhola

This comment has been minimized.

Contributor

davidpodhola commented Sep 24, 2018

I will support the new implementation by @navinkumarr.
Not only it is using a different database, but also using a different framework; Rocket is nice (note: we did not use it because we were told it is old and un-updated).
Also please note our implementation was done when we were very Microsoft oriented (everything in the repo is Windows-oriented); I hope the new one is multiplatform.
I think @navinkumarr you could at least reuse the tests we did, we had a very good code coverage.

@navinkumarr

This comment has been minimized.

navinkumarr commented Sep 25, 2018

@davidpodhola Thank you for reviewing the project. Your feedback is much appreciated. Will surely take reference from your application wherever possible.

@anishkny

This comment has been minimized.

Member

anishkny commented Sep 25, 2018

Go for it @navinkumarr best luck! Would be good if you could document differences from the first repo.

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