Reorder the exception fields to have the data field before the exception field

[ykitamura-mdsol]

As the title says

It's going to be a little slower but I think it's acceptable

Comparison:
              before:  3125192.9 i/s
               after:  2903105.3 i/s - same-ish: difference falls within error

[ejinotti-mdsol]

since you mentioned performance in the description, what about just the naive approach instead of compact?

exception_fields = {}
exception_fields[DATA] = param_data if param_data
exception_fields[EXCEPTION] = "#{exception.class}: #{exception.message}"
exception_fields[STACK] = backtrace

[ejennings-mdsol]

Hah, yeah, I was thinking of that or:

        exception_fields = {
          DATA => param_data,
          EXCEPTION => "#{exception.class}: #{exception.message}",
          STACK => backtrace
        }
        exception_fields.delete(DATA) unless param_data

[ykitamura-mdsol]

it's slow

Comparison:
              before:  3252707.9 i/s
               after:  3001680.9 i/s - same-ish: difference falls within error
               naive:  2732396.6 i/s - same-ish: difference falls within error