oracle-enhanced icon indicating copy to clipboard operation
oracle-enhanced copied to clipboard

Support for Rails 7.2

Open dwarner opened this issue 1 year ago • 19 comments

I know 7.1.0 just got released with support for Rails 7.1.x (thanks), but is there an ETA for Rails 7.2 support?

https://github.com/rsim/oracle-enhanced/blob/v7.1.0/activerecord-oracle_enhanced-adapter.gemspec s.add_runtime_dependency("activerecord", ["~> 7.1.0"])

I know Rails 7.2 ActiveRecord has a lot of "Remove deprecated" . . . https://guides.rubyonrails.org/7_2_release_notes.html#active-record

dwarner avatar Oct 04 '24 17:10 dwarner

+1

numbcoder avatar Oct 21 '24 09:10 numbcoder

+1

matthewkrom avatar Dec 13 '24 13:12 matthewkrom

+1. Any updates on this or any sort of plans to maintain this gem?

rjschof avatar Jan 17 '25 20:01 rjschof

~~I started attempting to upgrade to 7.2, and got stuck. https://github.com/rsim/oracle-enhanced/pull/2422 Anyone willing to give my very limited progress a look, I'd appreciate additional perspectives and/or suggestions for diagnosing.~~

andynu avatar Jan 22 '25 21:01 andynu

I've made further progress and cut a new PR for consideration. Not all the way there, but a significant step closer. There are now only four remaining test failures locally. #2424

andynu avatar Jan 31 '25 19:01 andynu

@andynu Thanks for your hard work!

We've pulled your 7.2 branch into one of our projects and have NOT run into a problem.

We'll be trying it on other projects coming up.

Do you have an ETA on an official release?

dwarner avatar May 15 '25 13:05 dwarner

Hi Dennis, I'm glad the patch has been working well for you, and me too! Alas, I am just a random user; I have no access to make releases.

On Thu, May 15, 2025 at 9:53 AM Dennis Warner @.***> wrote:

dwarner left a comment (rsim/oracle-enhanced#2411) https://github.com/rsim/oracle-enhanced/issues/2411#issuecomment-2883898490

@andynu https://github.com/andynu Thanks for your hard work!

We've pulled your 7.2 branch into one of our projects and have NOT run into a problem.

We'll be trying it on other projects coming up.

Do you have an ETA on an official release?

— Reply to this email directly, view it on GitHub https://github.com/rsim/oracle-enhanced/issues/2411#issuecomment-2883898490, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAONGOKA4PBXVOD5VMUTKL26SL4PAVCNFSM6AAAAABPMKZDQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQOBTHA4TQNBZGA . You are receiving this because you were mentioned.Message ID: @.***>

andynu avatar May 15 '25 13:05 andynu

Whoever tested this, maybe you add an approval so hopefully author attention is captured.

akostadinov avatar May 15 '25 14:05 akostadinov

@andynu Thanks for your hard work!

We've pulled your 7.2 branch into one of our projects and have NOT run into a problem.

We'll be trying it on other projects coming up.

Do you have an ETA on an official release?

+1 branch has been running in app ~6mo now. No issues. Thanks for all your work @andynu 🥳 (unfortunately, not able to approve)

joshRpowell avatar May 15 '25 14:05 joshRpowell

@andynu I will make a comment on the PR.

Have you gotten any responses/feedback from the maintainers?

dwarner avatar May 15 '25 16:05 dwarner

Nope

On May 15 2025, at 12:37 pm, Dennis Warner @.***> wrote:

dwarner left a comment (rsim/oracle-enhanced#2411) (https://github.com/rsim/oracle-enhanced/issues/2411#issuecomment-2884442406)

@andynu (https://github.com/andynu) I will make a comment on the PR.

Have you gotten any responses/feedback from the maintainers? — Reply to this email directly, view it on GitHub (https://github.com/rsim/oracle-enhanced/issues/2411#issuecomment-2884442406), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AAAONGNDVP553PICVGFRSRT26S7GPAVCNFSM6AAAAABPMKZDQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQOBUGQ2DENBQGY). You are receiving this because you were mentioned.

andynu avatar May 15 '25 16:05 andynu

@rsim Are you still maintaining this project?

dwarner avatar May 15 '25 16:05 dwarner

Yes, thank you @andynu . I am testing your branch now and things are looking good. Hoping this can be merged soon.

smarks-1-ND avatar May 15 '25 17:05 smarks-1-ND

Thank you @andynu ! I have a quite complex gem that connects to different databases.

Most of my tests worked out of the box with 7.2. Only one test failed. I am not sure if this is a Rails 7.2 behavior, a bug in the adapter, or if I do not know what I am doing.

In my gem, I have a method calling something like

SomeValue.where('value BETWEEN ? and ?', min, max)

The values for min and max come from another DB call. In 7.1 it works, in 7.2 I get the error ActiveRecord::StatementInvalid: OCIError: ORA-01722: invalid number

The issue is that min and max can be a BigDecimal (e.g. 0.23255e3).

The solution was to add .to_f to the variables.

SomeValue.where('value BETWEEN ? and ?', min.to_f, max.to_f)

Bjoernsen avatar May 16 '25 10:05 Bjoernsen

Hello, I tried @andynu fork on my little application connected to an old Oracle 11 DB. It worked without any problem after the update of the application to Rails 7.2.2.1.

Hope that a new release come out soon! Cheers

net1957 avatar May 16 '25 19:05 net1957

I upgraded one of my biggest Rails app (32,000 LOC) to 7.2 using your branch. All tests succeeded and up to now, I cannot find anything, that does not work.

Bjoernsen avatar May 19 '25 07:05 Bjoernsen

@rsim Are you still maintaining this project?

@dwarner It appears that this repo is mostly maintained by @yahonda. I would love to see @andynu added as a maintainer as someone who managed to contribute a lot to this Gem recently (if that's something they're interested in).

matthewtusker avatar May 29 '25 09:05 matthewtusker

+1. Any updates on this or any sort of plans to maintain this gem?

@yahonda Are you still actively maintaining this gem?

dwarner avatar May 29 '25 10:05 dwarner

@rsim Are you still maintaining this project?

@dwarner It appears that this repo is mostly maintained by @yahonda. I would love to see @andynu added as a maintainer as someone who managed to contribute a lot to this Gem recently (if that's something they're interested in).

I second that!

If we don't get a response, even though nothing has been posted since 2021, I might try pinging @yahonda here: https://groups.google.com/g/oracle-enhanced

dwarner avatar May 29 '25 16:05 dwarner

Hi all — pinging this thread since it's been two weeks.

Also, security support for Rails 7.1 (the highest version supported by this gem, aside from forks) ends in 3 months. "Active Support" by the rails team actually ended 3 months ago.

matthewkrom avatar Jun 12 '25 19:06 matthewkrom

If a few contributors agree, maybe we can make a fork (hopefully temporary until @rsim returns). I say a few so we are not stuck again with a single point of contact and have a basic form of peer review before merging. I think better to avoid a personal fork.

akostadinov avatar Jun 13 '25 11:06 akostadinov

https://github.com/rsim/oracle-enhanced/pull/2424 merged, only new gem version not released yet.

Update: Now released as https://github.com/rsim/oracle-enhanced/releases/tag/v7.2.0.rc1

Faq avatar Jun 13 '25 15:06 Faq

We've added 7.2.0.rc1 to a test environment. So far, no issues to report.

matthewkrom avatar Jun 20 '25 12:06 matthewkrom

Oracle enhanced adapter 7.2.0 has been released. Thanks for your patience. https://rubygems.org/gems/activerecord-oracle_enhanced-adapter/versions/7.2.0

yahonda avatar Jun 23 '25 13:06 yahonda

Closing this issue as Oracle enhanced adapter 7.2.0 has been released.

yahonda avatar Jun 23 '25 13:06 yahonda

Anybody interested in future Rails support, please take a look at #2471 and provide feedback, sorry for the spam but I think it deserves some wider attention.

akostadinov avatar Jul 25 '25 15:07 akostadinov