-
Notifications
You must be signed in to change notification settings - Fork 386
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
Rails 5.1 Breaks postgres adapter #780
Comments
well ar 5.0 (still) isnt really supported/tested much ... as far as I know. thus not sure if there was smt to break :) but thanks for the report. recommend you try take a look at it yourself as there isnt much activity besides patch merging/maintenance around here. |
I change this line in file arjdbc/jdbc.rb, please review this: rails-5...bencarsal:patch-1 |
@kares Can you help me understand why this is not a serious issue? I may be missing something, but doesn't it make JRuby unusable with Rails 5.1 and Postgres? |
@keithrbennett simply because there's no one to do it (personally trying to do maintenance of the stable line and some house-keeping/perf.work on master when I have time, the rest of JRuby team already tried doing the AR 5.x update but did not yet deliver fully) ... there are attempts but its a task that needs focused men power (AR-JDBC supports a lot of adapters and also accumulated some debt). have tried asking for support to speed things up (personally have no need to chase the Rails 5.x dragon yet) but no one cared much. simply said this is OSS. its a bit demotivating explaining this over-and-over (esp. when someone expects volunteers to have a certain sense of "seriousness") and communicating false hopes over private e-mail with occasional "supporters" that always go silent 😥 |
Firstly, thank you for the time and effort you've contributed. It's totally understandable that you have your own life and priorities, and are already contributing heavily, and no one can fault you for that. I was not complaining, I was just confused because I inferred that you believed that the issue was not important. Now I understand that you are not questioning the importance but just explaining that there is not enough manpower to address the issue. Thanks also for the explanation. |
I'm just curious as to why the rails team feels the need to change everything on each revision. Like sure, I read the rails doctrine and agree with their "Progress over stability" moto. However in the real world its painful that they change everything. The as400 system at my workplace has been around longer than I've been alive but since everything is backwards compatible, we don't have to worry about updates breaking stuff. Trying to explain to our boss why it takes us so much time and effort to upgrade rails is always an uphill battle |
See jruby/activerecord-jdbc-adapter#780. Signed-off-by: Pierre-Alexandre Meyer <[email protected]>
How can I help to support rails 5? I'm willing to help, but don't want to mess with java if possible =P |
This is sort of represented by our new label and targetted milestone but I left it open to resolve when 51.0 is released. |
Could it be that RoR 5.1 doesn't work with postgresl > 10 ? I found this page due to the fact that my rspecs starting to fail with a PG 10 database. They work fine with PG 9.5.10 version. |
The original issue is happening because the gem isn't compatible with Rails 5.1 yet. It should be fixed with this commit: 3ad4910 |
@rdubya feel free to close it - if you feel like there isn't much value here for the (pending) 5.1 work here |
Sounds good. I'll close it for now, it should be fixed with the 51 release. |
I'm still encountering this issue; I see there are 51.0 versions released for gems activerecord-jdbc-adapter, activerecord-jdbcmysql-adapter and activerecord-jdbcsqlite3-adapter, but activerecord-jdbcpostgresql-adapter is still on 50.0 - would it be possible to get a 51.0 release for the Postgres adapter, or is something else holding it up? Thank you! |
@abevoelker there was a lot of broken tests from 50 to 51 on postgresql. I think several of those have been fixed now so we should consider that even if it is not perfect. |
@rdubya @enebo this issue still persists, right @abevoelker ? Why the 51 gem is not yet available? Thanks a lot for your work |
@rdubya ++ for the patch ... it almost worked for me but I was also getting a
error. I just
in lib/arjdbc/postgresql/adapter.rb and all seems wll. |
@risinglf I'm currently (slowly) trying to patch up some more issues with 50 that I'm going to port over to 51 and then we'll likely be able to release a 51.0 for postgres. @michaeldbarry I haven't seen that issue, what version of rails and adapter gem are you using? Could you make a new issue for it so I can keep track of it a little easier? |
Hi, Is this the same issue? |
Hi @bgmat, we still haven't released a version of the postgres adapter that works with 5.1 yet, we are hopefully getting close though. |
Hi there! Any news? |
I get this error on startup of my server after upgrading to rails 5.1
I presume its from an activerecord change somewhere
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: