rsim / oracle-enhanced

Oracle enhaced adapter for ActiveRecord
MIT License
546 stars 308 forks source link

Add allow_retry kwarg to execute (7) #2374

Closed andynu closed 1 month ago

andynu commented 6 months ago

Add allow_retry kwarg to execute

This extends the existing class level auto_retry functionality to optionally allow an allow_retry boolean kwarg to allow retry of specific queries.

This PR preserves the requirement that autocommit? be enabled, which is the safer choice.

  def with_retry(allow_retry: false) # :nodoc:
    should_retry = (allow_retry || self.class.auto_retry?) && autocommit?

See issue #2310

Stacked on #2372