Xavier Noria
755af49755
edit pass to apply API guideline wrt the use of "# =>" in example code
2010-07-30 02:30:04 +02:00
Aaron Patterson
53fbbf4b83
fisting warning
2010-07-25 16:28:21 -07:00
José Valim
ff0d842454
Revert the previous three commits.
...
* 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;
2010-07-25 20:46:42 +02:00
Aaron Patterson
b7e0408ca9
use a hash to collect optional statistics about the instrumentation
2010-07-25 11:11:23 -07:00
Carlhuda
ba8d89c4c8
Performance optimizations to handle cases of instrumentors that are not listened to. Also, fix a possible concurrency issue.
2010-07-22 11:36:16 -07:00
Aaron Patterson
b2c8a5fd3e
Notifier API == Fanout API, so replace Notifier with Fanout as they quack the same
2010-07-18 16:49:29 -07:00
Aaron Patterson
2cbef6996c
bind method is not needed, so goodbye! <3 <3 <3
2010-07-18 15:37:23 -07:00
Hongli Lai (Phusion)
6f5fb13928
Remove documentation reference to AS::Notifications::Event#result because it has been removed.
2010-06-17 15:47:37 +02:00
Evgeniy Dolzhenko
ccf9577aee
Fix a bunch of minor spelling mistakes
2010-06-11 14:15:34 +04:00
Carlhuda
fc0882ba5a
Optimize AS::Notifications to remember which subscribers don't match and not run them. This will allow notifications that are only useful in dev or testing to run efficiently in production.
2010-03-01 17:45:37 -08:00
Carlhuda
c88360ef36
You can unsubscribe a subscriber
2010-03-01 17:45:37 -08:00
Prem Sichanugrist
f0523f72b4
Rename Rails::Subscriber to Rails::LogSubscriber
2010-02-16 22:36:15 +01:00
José Valim
31248fe369
Remove instrument! and require thread from AS::Notifications.
2010-02-04 10:39:55 +01:00
José Valim
0334f9f6cf
Add ActionDispatch::Notifications middleware.
2010-01-17 11:29:51 +01:00
José Valim
7f4d8e3fbd
Yield the payload notifications for further modification (like adding the result).
2010-01-15 12:24:31 +01:00
José Valim
7c3573f327
Add instrument! to notifications which adds the result to the payload.
2010-01-14 01:07:03 +01:00
José Valim
74f6ccea20
instrumenter should be accessible from ActiveSupport::Notifications.
2010-01-06 22:24:16 +01:00
Jeremy Kemper
4f2a04cc08
Notifications: extract central Notifier, cordon off the internal Fanout implementation, and segregate instrumentation concerns
2009-11-28 12:50:09 -08:00
Jeremy Kemper
02893d1705
Remark that Listener is an implementation detail
2009-11-28 12:50:09 -08:00
Jeremy Kemper
ddf681ce1d
Expose a simple Queue#wait to block until all notifications are drained
2009-11-28 12:50:09 -08:00
Jeremy Kemper
6f7fc5824f
Revert "Create SyncListener. Since they do not rely on Thread, they can be used on Google App Engine."
...
Take a step back on this API direction.
This reverts commit 8104f65c32
2009-11-28 12:50:09 -08:00
José Valim
8104f65c32
Create SyncListener. Since they do not rely on Thread, they can be used on Google App Engine.
...
Signed-off-by: Yehuda Katz <wycats@Yehuda-Katz.local>
2009-11-23 09:08:17 -08:00
Jeremy Kemper
58c0d31487
Notifications: queue.drained? for testability in place of brittle sleeps
2009-11-13 20:58:38 -08:00
Jeremy Kemper
3039912886
Silence spurious warning
2009-11-09 05:09:04 -08:00
Yehuda Katz
c9487ed6af
Change Event#thread_id to #transaction_id. Defaults to one "transaction" per thread but you can explicitly declare the start of a new one. This makes it possible for each request to have it own id.
2009-10-28 01:58:33 -07:00
Yehuda Katz
03d3824d96
Make it possible to have IDs per request
2009-10-28 01:50:59 -07:00
Yehuda Katz
cbcb947b00
AS::Notifications.subscribe blocks are now yielded the arguments to pass to AS::Notifications::Event.new
2009-10-27 21:04:53 -07:00
Jeremy Kemper
b30294b54a
Fix broken tests
2009-10-27 09:34:17 -07:00
Yehuda Katz
b3a198041b
Some optimizations on AS::Notifications. This does not change the public-facing API.
2009-10-27 09:15:41 -07:00
José Valim
2d7abe245e
Renamed Orchestra to Notifications once again [ #3321 state:resolved]
2009-10-15 18:51:51 -03:00
José Valim
8b340ab2f6
Revert "Rename Orchestra to Notifications [ #3321 state:resolved]"
...
This reverts commit 8cbf825425
2009-10-15 18:06:15 -03:00
Joshua Peek
8cbf825425
Rename Orchestra to Notifications [ #3321 state:resolved]
2009-10-14 19:50:36 -05:00
