# Adding the \`.rom\` suffix to emitted events

**URL:** <https://discourse.rom-rb.org/t/adding-the-rom-suffix-to-emitted-events/599>\
**Category:** Issues\
**Created:** [March 2, 2023, 9:18am UTC](https://discourse.rom-rb.org/t/adding-the-rom-suffix-to-emitted-events/599 "2023-03-02T09:18:24Z")\
**Posts on this page:** 4\
**Page:** 1

<div class="post-metadata">

**Author:** ![luismiramirez](https://yyz1.discourse-cdn.com/flex029/user_avatar/discourse.rom-rb.org/luismiramirez/32/292_2.png) [@luismiramirez](https://discourse.rom-rb.org/u/luismiramirez)\
**Post date:** [March 2, 2023, 9:18am UTC](https://discourse.rom-rb.org/t/adding-the-rom-suffix-to-emitted-events/599/1 "2023-03-02T09:18:24Z")

</div>

Hey there!

Would it be possible to add the `.rom` suffix to the events emitted through the instrumentation plugin? It’s how `ActiveRecord` works with `ActiveSupport::Notifications` and it’ll make things easier to identify different `sql` emitted events.

If it’s possible I’m happy to open a PR with the changes: [Rename emitted events to "sql.rom" · luismiramirez/rom-sql@6aa64c5 · GitHub](https://github.com/luismiramirez/rom-sql/commit/6aa64c5f47f255b9ea1c1d323b6cca4f714a93a7)

Thanks!

---

<div class="post-metadata">

**Author:** ![solnic](https://yyz1.discourse-cdn.com/flex029/user_avatar/discourse.rom-rb.org/solnic/32/279_2.png) [@solnic](https://discourse.rom-rb.org/u/solnic)\
**Post date:** [March 3, 2023, 11:14am UTC](https://discourse.rom-rb.org/t/adding-the-rom-suffix-to-emitted-events/599/2 "2023-03-03T11:14:29Z")

</div>

If you could make it configurable, then a PR would be great. It should default to `:sql` for backward-compatibility reasons.

---

<div class="post-metadata">

**Author:** ![luismiramirez](https://yyz1.discourse-cdn.com/flex029/user_avatar/discourse.rom-rb.org/luismiramirez/32/292_2.png) [@luismiramirez](https://discourse.rom-rb.org/u/luismiramirez)\
**Post date:** [March 3, 2023, 11:47am UTC](https://discourse.rom-rb.org/t/adding-the-rom-suffix-to-emitted-events/599/3 "2023-03-03T11:47:38Z")

</div>

Sure! I’ll work on it. Thank you!

---

<div class="post-metadata">

**Author:** ![luismiramirez](https://yyz1.discourse-cdn.com/flex029/user_avatar/discourse.rom-rb.org/luismiramirez/32/292_2.png) [@luismiramirez](https://discourse.rom-rb.org/u/luismiramirez)\
**Post date:** [March 6, 2023, 8:16am UTC](https://discourse.rom-rb.org/t/adding-the-rom-suffix-to-emitted-events/599/4 "2023-03-06T08:16:41Z")

</div>

Something that just came to my mind that I want to ask.

Since the next `rom-sql` release is going to be a major version bump, wouldn’t it be reasonable if the events are emitted with the suffix by default?
