Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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. | ||||
* | Fix a bunch of minor spelling mistakes | Evgeniy Dolzhenko | 2010-06-11 | 1 | -1/+1 |
| | |||||
* | Optimize AS::Notifications to remember which subscribers don't match and not ↵ | Carlhuda | 2010-03-01 | 1 | -1/+1 |
| | | | | run them. This will allow notifications that are only useful in dev or testing to run efficiently in production. | ||||
* | You can unsubscribe a subscriber | Carlhuda | 2010-03-01 | 1 | -0/+4 |
| | |||||
* | Rename Rails::Subscriber to Rails::LogSubscriber | Prem Sichanugrist | 2010-02-16 | 1 | -2/+2 |
| | |||||
* | Remove instrument! and require thread from AS::Notifications. | José Valim | 2010-02-04 | 1 | -1/+1 |
| | |||||
* | Add ActionDispatch::Notifications middleware. | José Valim | 2010-01-17 | 1 | -1/+1 |
| | |||||
* | Yield the payload notifications for further modification (like adding the ↵ | José Valim | 2010-01-15 | 1 | -1/+1 |
| | | | | result). | ||||
* | Add instrument! to notifications which adds the result to the payload. | José Valim | 2010-01-14 | 1 | -1/+1 |
| | |||||
* | instrumenter should be accessible from ActiveSupport::Notifications. | José Valim | 2010-01-06 | 1 | -8/+6 |
| | |||||
* | Notifications: extract central Notifier, cordon off the internal Fanout ↵ | Jeremy Kemper | 2009-11-28 | 1 | -132/+16 |
| | | | | implementation, and segregate instrumentation concerns | ||||
* | Remark that Listener is an implementation detail | Jeremy Kemper | 2009-11-28 | 1 | -1/+2 |
| | |||||
* | Expose a simple Queue#wait to block until all notifications are drained | Jeremy Kemper | 2009-11-28 | 1 | -2/+7 |
| | |||||
* | Revert "Create SyncListener. Since they do not rely on Thread, they can be ↵ | Jeremy Kemper | 2009-11-28 | 1 | -58/+34 |
| | | | | | | | | used on Google App Engine." Take a step back on this API direction. This reverts commit 8104f65c3225453d13307c3c2733c2a8f99e491a. | ||||
* | Create SyncListener. Since they do not rely on Thread, they can be used on ↵ | José Valim | 2009-11-23 | 1 | -34/+58 |
| | | | | | | Google App Engine. Signed-off-by: Yehuda Katz <wycats@Yehuda-Katz.local> | ||||
* | Notifications: queue.drained? for testability in place of brittle sleeps | Jeremy Kemper | 2009-11-13 | 1 | -9/+7 |
| |