[PUWEN-1504]: avrogen: optional maps

[cottinisimone]

https://prima-assicurazioni-spa.myjetbrains.com/youtrack/issue/PUWEN-1504

This is the easiest solution to fix this warning. I think we should remove this function here and let the team that uses this tool fix their test errors.

[oxg558]

let the team that uses this tool fix their test errors

That would be us :D

We'll have to see how much work it will be, but I wouldn't want to merge and deploy this before we fix them, as that would mean we couldn't upgrade the library version (and get any other fixes) until we do that. I seem to remember last time we looked it was quite a big job, and we don't really have the bandwidth at the moment.

[cottinisimone]

let the team that uses this tool fix their test errors

That would be us :D

We'll have to see how much work it will be, but I wouldn't want to merge and deploy this before we fix them, as that would mean we couldn't upgrade the library version (and get any other fixes) until we do that. I seem to remember last time we looked it was quite a big job, and we don't really have the bandwidth at the moment.

Ok, do you think that it might makes sense that we try to do that job for you?
I was wondering if this is just a matter of broken tests or it might affect production..?

[oxg558]

@cottinisimone I've done the work for pricing in the PR below (not sure you can view uk-pricing), and we just need to do the same again for dataland. It will affect the data pipelines so we're waiting on a response from the uk analysts before we merge anything.

https://github.com/primait/uk-pricing/pull/815

[cottinisimone]

@cottinisimone I've done the work for pricing in the PR below (not sure you can view uk-pricing), and we just need to do the same again for dataland. It will affect the data pipelines so we're waiting on a response from the uk analysts before we merge anything.

https://github.com/primait/uk-pricing/pull/815

Nope, i cannot see it.

Thank you @oxg558 for your help :)

[oxg558]

We have merged our patches so this should be OK for us now. Thanks for waiting!