-
Notifications
You must be signed in to change notification settings - Fork 28
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
Incorporate JRuby bits into pg gem #6
Comments
The active record tests look good. Last time i ran it i had 6 failures and 2 errors (out of 3787 tests). Ideally I'd wait until I fix those failures, but I'm swamped and I also don't think they're blockers. The other thing I'd like to take a look at is performance. Running the ActiveRecord test suite on JRuby is many times slower than running it on MRI. I think I'll focus on that one as soon as I have time. Again, I don't feel either one of these problems is a blocker. |
Hi! When @headius mailed me back in November about this, I replied with some concerns about merging the two, and I still have some of the same concerns. I'll repost my reply here with some edits to reflect the changes in the jruby-pg codebase since then: I agree that whatever we come up with should make it as easy as possible for the user who wants to install a gem to talk to Postgres, so with that as our guide, I'm sure we can come up with some arrangement. A couple of questions/issues to think about:
While we have some significant things to figure out, I don't think any of it is insurmountable. I'm excited for JRuby users! @jvshahid's contribution is an impressive body of work, and I'm happy to help in whatever way I can to get it out there so folks can use it. I'm looking forward to working with you both. |
Things has been quiet for some time. So let me add my input on some of the open questions:
I hope I provided satisfactory answers to your questions. Looking forward to work with you as well. |
See ged/ruby-pg#1 |
Copying @ged, @jvshahid.
I think we're at or close to a point where we could release this as a -java version of the pg gem. There are still many libraries and applications that depend on functions the pg library provides that JDBC does not.
@ged Are you interested in working with us to get a -java version of the pg gem made? As I see it, the only requirements are the following:
We would obviously continue to support the Java version of the gem, and we do not expect you to take that over.
@jvshahid What remains before you would be comfortable releasing this? I know you had been looking into ActiveRecord tests when we paused a few months ago.
The text was updated successfully, but these errors were encountered: