-
-
Notifications
You must be signed in to change notification settings - Fork 747
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
SQLAlchemy support? #17
Comments
Hello! I'd love to add PostgreSQL support but it's not something I would feel confident doing right now as I don't work with that DB (not in Python at least). PostgreSQL support would be a fantastic opportunity for other contributors to jump in! There's a DataLayer interface in place already, ready to be abused. If somebody wants to help well, let's get in touch! |
I assume we can use flask-SqlAlchemy to kill several flies with one kill? |
Do you frequent any irc channels, skype, gtalk or something? So we can talk what would be involved in this. |
I'm interested, if only to help test. Not sure what I can do on the dev side. |
Here's a bounty - anyone that implements this properly gets a small VM at 6sync for three months! And you get to contribute to open source! |
I guess Flask-SqlAlchemy would work, although I'm not sure a full featured ORM is needed. Again, I'm out of my comfort zone here so I'm open to discussion, ideas and pull requests. I've been amusing myself with the idea of an irc channel for Eve, but it seemed a bit overkill at this stage. Also, by keeping the discussion here we allow everybody interested to easily jump in with his/her opinions and ideas. |
I'm all for irc channel, let's go! |
I too am very interested in collaborating with anyone working with/on On Tue, Feb 26, 2013 at 10:19 AM, Mario Đanić notifications@github.comwrote:
|
@inactivist hey man I'm sure you can lend a hand in one way or another! |
IRC channel is now on freenode #evehq :) |
the cat is out of the bag I guess |
Is it reasonable for SQL support to exclude |
I think supporting whatever sqlalchemy supports is fine :) On Friday, March 8, 2013, Michael Curry wrote:
|
@inactivist I guess we could get away with that, at least on initial release. Remember you can have your own custom data validator within the SQL subpackage (for reference, see the validator for the mongo subpackage). |
If someone would like to test/review - here is my initial implementation:
|
@Tefnet If someone else will not pickup your branch, I would like to try it and add tests to it this weekend! |
A quick note (haven't looked at the code yet). It seems kind of awkward that one has to define the schema twice, one for SQLAlchemy and another in |
I don't find that particularly strange due to the way Eve works - however, one could write a script to translate the SQLAlchemy models and spit out Cerberus validation rules for settings.py. |
I've made simple decorator for SQLAlchemy model Tefnet@bfb3798
|
https://gist.github.com/Tefnet/5430309 simple demo with decorator for https://github.com/nicolaiarocci/eve/issues/45 |
A WIP based on #45 related to SQLAlchemy tests: branch: https://github.com/kpanic/eve/commits/sqlalchemy (might need a rebase from #45 ) |
Guys, let's push this properly - I have time this week, so let's meet up on IRC and at least finish something worthy of a release? :) |
Guys, you alive? :) |
Ping. Has this gone anywhere? I would be interested in helping where I can. We might be able to use something like this at work so I could devote some time to getting SQLAlchemy up and running. |
Hi, are you on irc @nZac? Let's talk! Cheers. |
#python-eve right? |
Yup. On Mon, Jul 15, 2013 at 5:17 PM, Nick Zaccardi notifications@github.comwrote:
|
@nZac nZac around? :) |
This gist might help: |
What's the current status on this? Is only testing needed or is essential implementation work still missing? |
there's a guy on the mailing list who's willing to work on the branch and is looking for help. Another guy jumped in (via email). I'm still interested in merging as soon as it is mature enough so everybody is welcome to chime in (maybe by answering to that message on the ML in the first place). |
I have also started to work on my fork - https://github.com/RedTurtle/eve/tree/sqlalchemy based on @Tefnet branch. |
@amleczko this is very interesting, consider getting in touch with the guys on the mailing list so maybe you can join efforts. |
Hello everybody, I just merged a huge PR to the SQLAlchemy branch. It brings the functionality almost on par (with some tests and refactoring) with current Eve 0.4-dev so you may all want to work on it. Author's comment on the PR (#311)
For reference: |
All those interested, #321 (support for DELETE and lots of tests) was just merged to the SQLAlchemy branch. Things are moving forward rather quickly now and I'd really appreciate if someone would code-review and/or contribute to the sql branch. |
|
I'm the contributor to the merges to the SQL branch of Eve. I just applied to join the mailing list so we should be able to discuss this there soon. Meanwhile...in the past few weeks I focused on getting to near feature parity with eve 0.4-dev and more importantly in porting the relevant integration tests to SQL. I'm sure there is plenty of room for improvement in my implementation, especially for supporting features that are particular to SQL like composite primary keys. Here is a list of things I have planned for the immediate future or where people could focus:
|
👍 that would be also useful for elastic extension |
Hi. We are trying out the SQLAlchemy branch, and we're wondering if the filters feature are already integrated. If it is, we would like to know how they are invoked on the API (like how does the where query work?) If in case it's not yet here, we would love to help out to contribute code, and also we would like to know if you guys have any preference as to how the where query should look like. Cheers! |
Yes they are although only in the python form. I made no effort to support the Mongo query syntax.
where obviously |
Status update: Python 3.4 compatibility has just been added with PR #382. Cheers! |
Also see PR #424 which I find very interesting. |
Hello,
I was just wondering if you've got any idea when PostgreSQL is going to be supported?
Appreciate the work you're doing!
Cheers!
The text was updated successfully, but these errors were encountered: