Skip to content

Restore RubyHash(Ruby, int) for older ARJDBC versions#9456

Merged
headius merged 2 commits into
jruby:masterfrom
headius:restore_deprecated_rubyhash_constructor
Jun 13, 2026
Merged

Restore RubyHash(Ruby, int) for older ARJDBC versions#9456
headius merged 2 commits into
jruby:masterfrom
headius:restore_deprecated_rubyhash_constructor

Conversation

@headius

@headius headius commented May 22, 2026

Copy link
Copy Markdown
Member

This constructor is used by several current versions of ARJDBC. It was fixed for future versions in jruby/activerecord-jdbc-adapter#1210 but not backported to any of the earlier versions yet.

This constructor was restored by deprecation patches from 10.0 in
jruby#9474 but the merge of that to 10.1 left it as a normal
non-delegated constructor. This patch makes the following changes:

* Make this constructor delegate, like the other deprecated
  RubyHash constructors.
* Remove the protected constructor from RubyHashLinkedBuckets, so
  that it lives only in RubyHash and we can be sure all callers
  need delegation.

The constructor should move back into RubyHashLinkedBuckets as a
protected internal constructor once we can safely remove the
constructors from RubyHash (10.2 or later).
@headius
headius force-pushed the restore_deprecated_rubyhash_constructor branch from d02a478 to 0d9a4db Compare June 13, 2026 15:31
@headius
headius merged commit 04bd9a4 into jruby:master Jun 13, 2026
134 checks passed
@headius
headius deleted the restore_deprecated_rubyhash_constructor branch June 13, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant