Proposal: EventLogger shouldn't be a Role

Astron / Astron

A Server Technology for Realtime Object Networking

https://astron.github.io/astron

Other

201 stars 88 forks source link

Proposal: EventLogger shouldn't be a Role #270

Open kestred opened 10 years ago

kestred commented 10 years ago

EventLogger should have its own section in the config file and shouldn't really be a role.

It breaks the (not-really-required but common) pattern of interfacing directly with the MessageDirector.
Unlike other roles there shouldn't be multiple of ELs -- especially not multiple per daemon -- (Roles are all components which you add more of to help scale and to provide clusters, the EventLogger is intended for the most part to be a singleton, and further only used in development).

AlJaMa commented 10 years ago

Interesting take on the EV. I had been thinking about it in almost the exact opposite way. My thoughts are that the EL can be used in production for things like application and usability logging. For this purpose, it may make sense to have multiple event loggers in a single astron cluster (or possibly a single one that messages can route to). I do agree however that there is no need for multiple EL per daemon.