Change With*Unmarshallers signatures

[sincejune]

Description:

• Change method signatures as recommended in Kafka exporter/receiver change With*Unmarshallers signature #2967 (Closes Kafka exporter/receiver change With*Unmarshallers signature #2967)

[codecov]

Codecov Report

Merging #2973 (b72494b) into main (13e4566) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2973   +/-   ##
=======================================
  Coverage   91.63%   91.63%           
=======================================
  Files         312      312           
  Lines       15404    15404           
=======================================
  Hits        14116    14116           
  Misses        881      881           
  Partials      407      407           

Impacted Files	Coverage Δ
exporter/kafkaexporter/factory.go	100.00% <100.00%> (ø)
receiver/kafkareceiver/factory.go	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 13e4566...b72494b. Read the comment docs.

[sincejune]

The CI failure seems unrelated to my change. Could any try to re-run the CI?

[mx-psi]

We use 'Unmarshaler' (single 'l') on other places in the Collector while we use 'Unmarshallers' here. Both forms are valid English spelling but I think we should consider using a consistent spelling. Maybe this is a good PR to do so (if we want to use the single 'l' version at least)?

[sincejune]

This sounds good. I rollbacked my changes to the variables. I'd like to have another PR to rename Marshaller to Marshaler if you decide to do so.

(Also, please feel free to rollback my last commit if you choose the double ll version)

[bogdandrutu]

@sincejune @mx-psi good catch, I would prefer the one "l" version (less spelling).