Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #34707 from xlts/update-notifications-docs | Gannon McGibbon | 2018-12-31 | 1 | -2/+5 |
|\ | | | | | Add examples describing error handling in ActiveSupport::Notification… | ||||
| * | Add examples describing error handling in ActiveSupport::Notifications and ↵ | Mariusz Hausenplas | 2018-12-14 | 1 | -2/+5 |
| | | | | | | | | ActiveSupport::LogSubscriber documentation files | ||||
* | | Extend documentation of `ActiveSupport::Notifications.subscribe` (#34721) | Bogdan | 2018-12-17 | 1 | -1/+8 |
|/ | | | | | | | | | | | * Extend documentation of `ActiveSupport::Notifications.subscribe` Add mention that a block with only one argument passed to the method will yield an event object. Related to #33451 * Emphasize that `SubscribeEventObjects` is a test class by adding suffix `Test` | ||||
* | Add docs to ActiveSupport::Notifications.subscribe | Chris Fung | 2018-10-22 | 1 | -0/+18 |
| | | | | [ci skip] | ||||
* | clarify role of unique_id in ActiveSupport::Notifications | zvkemp | 2018-10-05 | 1 | -2/+2 |
| | |||||
* | [Active Support] require_relative => require | Akira Matsuda | 2017-10-21 | 1 | -3/+3 |
| | | | | This basically reverts 8da30ad6be34339124ba4cb4e36aea260dda12bc | ||||
* | [Active Support] `rubocop -a --only Layout/EmptyLineAfterMagicComment` | Koichi ITO | 2017-07-11 | 1 | -0/+1 |
| | |||||
* | Use frozen-string-literal in ActiveSupport | Kir Shatrov | 2017-07-09 | 1 | -0/+1 |
| | |||||
* | [Active Support] require => require_relative | Akira Matsuda | 2017-07-01 | 1 | -3/+3 |
| | |||||
* | Add error logging to Active Job | Steven Bull | 2017-03-27 | 1 | -0/+2 |
| | | | | | | | | | | | | | | | | | | | | Active Job logging instrumentation is changed to log errors (with backtrace) when a job raises an exception in #perform. This improves debugging during development and test with the default configuration. Prior to Rails 5, the default development configuration ran jobs with InlineAdapter, which would raise exceptions to the caller and be shown in the development log. In Rails 5, the default adapter was changed to AsyncAdapter, which would silently swallow exceptions and log a "Performed SomeJob from Async..." info message. This could be confusing to a developer, as it would seem that the job was performed successfully. This patch removes the "Performed..." info message from the log and adds an error-level "Error performing SomeJob..." log message which includes the exception backtrace for jobs that raise an exception within the #perform method. It provides this behavior for all adapters. | ||||
* | stop using removed `render :text` | yuuji.yaginuma | 2016-12-03 | 1 | -2/+2 |
| | | | | Follow up to 79a5ea9eadb4d43b62afacedc0706cbe88c54496 | ||||
* | applies new string literal convention in activesupport/lib | Xavier Noria | 2016-08-06 | 1 | -3/+3 |
| | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
* | [ci skip] Add description about which object | yui-knk | 2015-09-02 | 1 | -2/+2 |
| | | | | `ActiveSupport::Notifications.subscribe` expects as second parameter. | ||||
* | added description for rails generators, and fixed sentence formation for ↵ | Rishi Jain | 2014-11-10 | 1 | -1/+1 |
| | | | | active_support/notifications [ci skip] | ||||
* | Update documentation to reflect unsubscription with name. | Guo Xiang Tan | 2014-06-01 | 1 | -2/+7 |
| | |||||
* | Stop using method missing for singleton delegation. | Aaron Patterson | 2013-11-06 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This saved about 46 array allocations per request on an extremely simple application. The delegation happened in the notification subsystem which is a hotspot, so this should result in even more savings with larger apps. Squashed commit of the following: commit 41eef0d1479526f7de25fd4391d98e61c126d9f5 Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Wed Nov 6 16:32:31 2013 -0800 speed up notifications commit 586b4a18656f66fb2c518fb8e8fee66a016e8ae6 Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Wed Nov 6 16:31:05 2013 -0800 speed up runtime registry methods commit b67d074cb4314df9a88438f785868cef77e583d7 Author: Aaron Patterson <aaron.patterson@gmail.com> Date: Wed Nov 6 16:28:12 2013 -0800 change method name and make it public | ||||
* | Documentation: Notifications queue does not run in a thread. | David Butler | 2013-05-20 | 1 | -2/+2 |
| | |||||
* | hides the per thread registry instance, and caches singleton methods | Xavier Noria | 2013-04-13 | 1 | -4/+0 |
| | | | | | | | | | | | | Existing code was delegating to the instance with delegate macro calls, or invoking the instance method to reach the object and call its instance methods. But the point is to have a clean class-level interface where the thread local instance is hidden in the implementation. References #11c6973. References #10198. | ||||
* | Changed the ScopeRegistry and the InstrumentationRegistry to use the | wangjohn | 2013-04-09 | 1 | -5/+4 |
| | | | | PerThreadRegistry module. | ||||
* | Creating an object to consolidate thread locals which hold the | wangjohn | 2013-04-08 | 1 | -1/+27 |
| | | | | instrumenters for the AS::Notifications module. | ||||
* | 1.9 hash syntax in activesupport | AvnerCohen | 2012-11-10 | 1 | -9/+9 |
| | |||||
* | Improve documentation for subscribe block | Grant Hutchins & Sabrina Staedt | 2012-09-27 | 1 | -1/+11 |
| | |||||
* | update AS docs [ci skip] | Francesco Rodriguez | 2012-09-17 | 1 | -1/+2 |
| | |||||
* | update some AS code examples to 1.9 hash syntax [ci skip] | Francesco Rodriguez | 2012-09-12 | 1 | -8/+8 |
| | |||||
* | move fanout back to a global variable, add a mutex for safety | Aaron Patterson | 2012-06-19 | 1 | -26/+5 |
| | |||||
* | remove global cache | Aaron Patterson | 2012-06-19 | 1 | -7/+2 |
| | |||||
* | reduce thread locals | Aaron Patterson | 2012-06-19 | 1 | -3/+20 |
| | |||||
* | make the fanout notifier local to the current thread | Aaron Patterson | 2012-06-19 | 1 | -4/+8 |
| | |||||
* | Documentation: make it clearer that subscribers are not notified ↵ | Schnittchen | 2012-06-16 | 1 | -1/+1 |
| | | | | asynchronously, but saved for later use. | ||||
* | Add documentation to detail passing of an object as the second | Michael de Silva | 2012-04-13 | 1 | -0/+27 |
| | | | | | | | parameter passed to the ActiveSupport::Notifications.subscribe method instead of a block Example code sample and output is provided as well. | ||||
* | Fix further typos in ActiveSupport::Notifications | Michael de Silva | 2012-04-13 | 1 | -4/+4 |
| | |||||
* | Fix rdoc typo in ActiveSupport::Notifications | Michael de Silva | 2012-04-13 | 1 | -1/+1 |
| | |||||
* | remove duplicate requires. thanks @atambo | Aaron Patterson | 2012-03-21 | 1 | -1/+0 |
| | |||||
* | push the autoloads up to requires | Aaron Patterson | 2012-03-21 | 1 | -4/+4 |
| | |||||
* | warns against using temporary subscribers | Xavier Noria | 2012-01-12 | 1 | -0/+4 |
| | |||||
* | fixes typo | Xavier Noria | 2011-11-05 | 1 | -1/+1 |
| | |||||
* | implements AS::Notifications.subscribed, which provides subscriptions to ↵ | Xavier Noria | 2011-11-05 | 1 | -0/+41 |
| | | | | events while a block runs | ||||
* | expands the documentation of AS::Notifications | Xavier Noria | 2011-11-05 | 1 | -14/+46 |
| | |||||
* | Removing unnecessary require, solve 'circular require considered harmful' ↵ | Jon Leighton | 2011-08-16 | 1 | -2/+0 |
| | | | | warning. | ||||
* | fanout unsubscribe only accepted one argument, so taking *args here is ↵ | Aaron Patterson | 2011-02-09 | 1 | -2/+2 |
| | | | | probably bad | ||||
* | just use an attr_accessor so we do not pay ||= on every notification call | Aaron Patterson | 2011-02-09 | 1 | -5/+3 |
| | |||||
* | speed up notification publishing by writing the delegate method | Aaron Patterson | 2011-02-09 | 1 | -1/+4 |
| | |||||
* | edit pass to apply API guideline wrt the use of "# =>" in example code | Xavier Noria | 2010-07-30 | 1 | -3/+3 |
| | |||||
* | fisting warning | Aaron Patterson | 2010-07-25 | 1 | -1/+1 |
| | |||||
* | Revert the previous three commits. | José Valim | 2010-07-25 | 1 | -13/+3 |
| | | | | | | * AS::Notifications#instrument should not measure anything, it is not its responsibility; * Adding another argument to AS::Notifications#instrument API needs to be properly discussed; | ||||
* | use a hash to collect optional statistics about the instrumentation | Aaron Patterson | 2010-07-25 | 1 | -3/+13 |
| | |||||
* | Performance optimizations to handle cases of instrumentors that are not ↵ | Carlhuda | 2010-07-22 | 1 | -2/+22 |
| | | | | listened to. Also, fix a possible concurrency issue. | ||||
* | Notifier API == Fanout API, so replace Notifier with Fanout as they quack ↵ | Aaron Patterson | 2010-07-18 | 1 | -23/+1 |
| | | | | the same | ||||
* | bind method is not needed, so goodbye! <3 <3 <3 | Aaron Patterson | 2010-07-18 | 1 | -1/+1 |
| | |||||
* | Remove documentation reference to AS::Notifications::Event#result because it ↵ | Hongli Lai (Phusion) | 2010-06-17 | 1 | -1/+0 |
| | | | | has been removed. |