| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|/
|
|
| |
Backport test from #14410
|
|
|
|
|
| |
Previously, any of the connection checkouts could have failed, and this
test would pass.
|
| |
|
|\
| |
| | |
Track owning thread for connection pool
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
.. not a general timeout.
Now, if a thread checks out a connection then dies, we can immediately
recover that connection and re-use it.
This should alleviate the pool exhaustion discussed in #12867. More
importantly, it entirely avoids the potential issues of the reaper
attempting to check whether connections are still active: as long as the
owning thread is alive, the connection is its business alone.
As a no-op reap is now trivial (only entails checking a thread status
per connection), we can also perform one in-line any time we decide to
sleep for a connection.
|
|/
|
|
| |
It wasn't doing anything beyond clearing the statement cache.
|
|\
| |
| | |
Add Public Api for Register New Extensions for Rake Notes
|
| |
| |
| |
| | |
config/application.rb file
|
| | |
|
| | |
|
| |
| |
| |
| | |
at config level
|
| | |
|
| | |
|
| |
| |
| |
| | |
have an API for register it in the corresponding gems
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
Fortisque/kevin/stream_error_in_main_thread_if_not_committed
re-raise error if error occurs before committing in streaming
|
| | |
| | |
| | |
| | | |
update the tests, using an if-else
|
| | |
| | |
| | |
| | | |
Closes #14406.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Closes #14405.
This is a follow-up to 9e997e9039435617b6a844158f5437e97f6bc107 to restore
the documented behavior.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
than assume SecureRandom is available
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The method `shallow?` returns false if the parent resource is a singleton so
we need to check if we're not inside a nested scope before copying the :path
and :as options to their shallow equivalents.
Fixes #14388.
|
| | |
| | |
| | |
| | | |
Follow up to 024e5e31
|
| | | |
|
|\ \ \
| | | |
| | | | |
Extend fixture label replacement to allow string interpolation
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Allows fixtures to use their $LABEL as part of a string instead
of limiting use to the entire value.
mark:
first_name: $LABEL
username: $LABEL1973
email: $LABEL@$LABELmail.com
users(:mark).first_name # => mark
users(:mark).username # => mark1973
users(:mark).email # => mark@markmail.com
|
|\ \ \
| | | |
| | | | |
Reword 5.6 strong parameters and private method stuff [ci skip]
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Fix MailerPreview broken tests
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
`BaseMailerPreview.welcome` is an instance method, so we need to stub the
method on a instance level and not on Class. The stub is important to make
sure the Message object is the same in the other expectations.
This was working randomly because Mocha uses == to compare two objects
on the `with()` expectation and even if the Mail::Message objects were
not the same object they are equal, but thats not the case in 100% of
the runs. So we need to make sure we use `.any_instance` method and have
the right message object.
|
|\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
add include_hidden option to collection_check_boxes helper
Conflicts:
actionview/CHANGELOG.md
actionview/test/template/form_collections_helper_test.rb
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | | |
Minor grammer, code conventions fix [ci skip]
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | | |
Conflicts:
activesupport/lib/active_support/core_ext/hash/conversions.rb
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
requires. Appropriate to keep this, users don't care that the implementation got unified.
|
|\ \ \ \ \
| |_|/ / /
|/| | | |
| | | | |
| | | | | |
tgxworld/use_teardown_helper_method_in_activerecord
Use teardown helper method.
|
| | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Follow-Up to https://github.com/rails/rails/pull/14348
Ensure that SQLCounter.clear_log is called after each test.
This is a step to prevent side effects when running tests. This will allow us to run them in random order.
|
| | | |
| | | |
| | | |
| | | | |
Fixes #14383.
|
|\ \ \ \
| | | | |
| | | | | |
Update callbacks executed on AR::Base#touch [skip ci]
|
| | |_|/
| |/| |
| | | |
| | | |
| | | | |
As of https://github.com/rails/rails/pull/12031 after_commit and
after_rollback are also executed
|
|\ \ \ \
| | | | |
| | | | | |
Fix #to_json for BasicObject Enumerables
|
| |/ / / |
|
|\ \ \ \
| | | | |
| | | | | |
Allow custom JDBC urls
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
mitigates #14323
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Before we had a bug in the resolver cache so the disable_cache were not
working when passing options to find
|
|\ \ \ \ \
| |_|/ / /
|/| | | |
| | | | | |
Ensure LookupContext in Digestor selects correct variant
|