Please privide Rails 7.1 compatibility

[drgcms]

Steps to reproduce

Database connection

Expected behavior

Success

Actual behavior

oracle-enhanced (deb214decc37) lib/active_record/connection_adapters/oracle_enhanced_adapter.rb:699:in get_database_version'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_adapters/schema_cache.rb:374:in database_version'
activerecord (7.1.2) lib/active_record/connection_adapters/schema_cache.rb:70:in database_version'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_adapters/schema_cache.rb:200:in database_version'
activerecord (7.1.2) lib/active_record/connection_adapters/abstract_adapter.rb:871:in database_version'](http://razvoj:3340/register-finance#) [oracle-enhanced (deb214decc37) lib/active_record/connection_adapters/oracle_enhanced_adapter.rb:307:in supports_fetch_first_n_rows_and_offset?'
oracle-enhanced (deb214decc37) lib/active_record/connection_adapters/oracle_enhanced_adapter.rb:271:in arel_visitor'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_adapters/abstract_adapter.rb:159:in initialize'
oracle-enhanced (deb214decc37) lib/active_record/connection_adapters/oracle_enhanced_adapter.rb:250:in initialize'](http://razvoj:3340/register-finance#) [oracle-enhanced (deb214decc37) lib/active_record/connection_adapters/oracle_enhanced_adapter.rb:76:in new'
oracle-enhanced (deb214decc37) lib/active_record/connection_adapters/oracle_enhanced_adapter.rb:76:in oracle_enhanced_connection'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_pool.rb:673:in public_send'
activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_pool.rb:673:in new_connection'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_pool.rb:720:in checkout_new_connection'
activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_pool.rb:699:in try_to_checkout_new_connection'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_pool.rb:657:in acquire_connection'
activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_pool.rb:355:in checkout'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_pool.rb:184:in connection'
activerecord (7.1.2) lib/active_record/connection_adapters/abstract/connection_handler.rb:246:in retrieve_connection'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/connection_handling.rb:287:in retrieve_connection'
activerecord (7.1.2) lib/active_record/connection_handling.rb:254:in connection'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/model_schema.rb:618:in load_schema!'
activerecord (7.1.2) lib/active_record/attributes.rb:264:in load_schema!'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/encryption/encryptable_record.rb:127:in load_schema!'
activerecord (7.1.2) lib/active_record/model_schema.rb:563:in block in load_schema'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/model_schema.rb:560:in synchronize'
activerecord (7.1.2) lib/active_record/model_schema.rb:560:in load_schema'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/model_schema.rb:441:in attribute_types'
activerecord (7.1.2) lib/active_record/model_schema.rb:467:in type_for_attribute'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/type_caster/map.rb:16:in type_for_attribute'
activerecord (7.1.2) lib/arel/table.rb:111:in type_for_attribute'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/table_metadata.rb:18:in type'
activerecord (7.1.2) lib/active_record/relation/predicate_builder.rb:59:in build'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/relation/predicate_builder.rb:54:in []'
activerecord (7.1.2) lib/active_record/relation/predicate_builder.rb:132:in block in expand_from_hash'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/relation/predicate_builder.rb:79:in each'
activerecord (7.1.2) lib/active_record/relation/predicate_builder.rb:79:in flat_map'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/relation/predicate_builder.rb:79:in expand_from_hash'
activerecord (7.1.2) lib/active_record/relation/predicate_builder.rb:25:in build_from_hash'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/relation/query_methods.rb:1526:in build_where_clause'
activerecord (7.1.2) lib/active_record/relation/query_methods.rb:943:in where!'](http://razvoj:3340/register-finance#) [activerecord (7.1.2) lib/active_record/relation/query_methods.rb:938:in where'
activerecord (7.1.2) lib/active_record/querying.rb:23:in where'](http://razvoj:3340/register-finance#) [/home/rails/register/app/renderers/register_renderer.rb:102:in finance'

System configuration

Rails version: rails (7.1.2)

Oracle enhanced adapter version: activerecord-oracle_enhanced-adapter (7.1.0.alpha deb214d)

Ruby version: ruby 3.2.2 (2023-03-30 revision e51014f9c0) [x86_64-linux]

Oracle Database version: 12.1.0.2.0

Thanks
TheR

[jarl-dk]

This request covers #2329

[andynu]

Hi Everyone,

I'm interested in helping move this forward. I've started a series of PRs to work through test regressions on the way to rails 7.1 support. Right now I'm providing them as stacked PRs so you can see the passing tests. I'd be happy to provide them again as independent PRs, each one really just contains a rails feature bump and a single commit of the fixing changes. Since there is no contribution guide here, I'm not sure if this is the most desirable approach. Please let me know if I can change tactics to make this easier on the maintainers.

I would appreciate any feedback you have. I'm trying to be as minimal as possible in the changes to make these tests pass, there are already scenarios where a larger change might be more appropriate. Thank you!!!

• Add _connection method (1) #2367
• Ensure raw_connection has a @raw_connection set (2) #2368
• Improve MockLogger - level/limited missing method (3) #2369
• Support Rails 7.1's Independent SchemaMigration object (4) #2370
• Support Rails 7.1 ColumnDefinition validations (5) #2371
• Rails 7.1 use the visitor pattern when building the in clause (6) #2372
• Add allow_retry kwarg to execute (7) #2374
• Override the max_index_name_size to keep local shortening algorithm. (8) #2373
• Fix deprecation warning about serializer keywords (9) #2375
• Add alias to internal_exec_query (11) #2376
• Switch to Arel::Table.name instead of removed alias table_name. (10) #2379
• "Assign auto populated columns on Active Record object creation", breaks the returning of id on insert #2380
  • Make insert return an array, and include the id_value if super returns nil (12) #2381

[davinlagerroos]

First, thanks @andynu for the work you've put in to upgrading this library!

Second, I am by no means an expert in this library or its patterns. My group uses this gem and I have been looking at a 7.1 update well, so I have some ideas about a couple of these changes that I will add to the relevant PRs. However, I am still orienting myself to the codebase so take whatever I say in those PRs with a grain of salt.

[joshRpowell]

Anyone know how this recent change in Rails rails/rails@d0af407 impacts this Gem?

[andynu]

Josh, Some facts: - Rails's main branch has moved on to rails 8.0. - Rails's has branched 7-2-stable - Rails 7.1 released 7 months ago. - There are no official versions of this gem that supports or tests against any of those rails versions. What I take from that commit is just that the rails team itself is limiting the scope of what they are actively supporting. It does not preclude other gems continuing to exist and integrate with active-record. But it does put the entire onus of support on those 3rd party gems. Said another way: We're on our own.

…

On Wed, May 15, 2024 at 10:12 AM Josh Powell ***@***.***> wrote: Anyone know how this recent change in Rails ***@***.*** <rails/rails@d0af407> impacts this Gem? — Reply to this email directly, view it on GitHub <#2357 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAONGJJLFKVIBNXCV6QHJDZCNULDAVCNFSM6AAAAABB3K7GNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGY3DEMJYHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[mohits]

I remember the discussion that support will be provided officially by Rails core for the major supported OSS databases. The discussion was in the context of DB2 support and is at this link: rails/rails#50537

In fact, this gem was specifically mentioned in the discussion. I read it as meaning that Rails core will officially support only those adapters but the community is welcome to add support for others and yes, the onus is on them and their users. However, I have no doubt that changes could be upstreamed to Rails if it were broadly applicable.

[jacobat]

@yahonda Are you aware of anyone working on providing a Rails 7.1 compatible version beyond what's mentioned in this thread? Are you still working on this adapter or do we need to find new maintainer(s)?

[yahonda]

Sorry for my late update. I still have some plan to support Oracle enhanced adapter compatible with Rails 7.1. That will take some more time.

[hss-mateus]

#2384 have all the patches mentioned above and a couple of more things

Unfortunatelly I don't have time to test it with different scenarios than a simple model

[rjschof]

This is a blocker for any app relying on Oracle DBs to get to Rails 7.1. It looks like the code in the PR #2384 works pretty well. What needs to be done to get this in a new release of this gem? How can I help?

[andynu]

@yahonda Thank you for all the merges! Anything you'd like to discuss or further changes you're interested in related to rails 7.x support, I'm here for it.

[dwarner]

I see the code has been merged into master (i.e. 7.1.0.alpha), but Is there an estimate on the official release date of 7.1.0?

[joshRpowell]

@dwarner a tagged beta is now available for testing https://github.com/rsim/oracle-enhanced/releases/tag/v7.1.0.beta1

[andynu]

I've switched to this from my fork. So far, so good. Tests are green. Nothing surprising yet. Thank you so much @yahonda for cutting the public release.

[Faq]

With release of v7.1, forgotten to update readme too? https://github.com/rsim/oracle-enhanced#installation

[yahonda]

Thank you for waiting for the release of Active Record Oracle enhanced adapter 7.1. The README has been updated via #2408 as well, so I am going to close this issue. If you encounter any problems, please feel free to open a new issue.