Services -changed from List to Set

[camunda]

Hi Alex,

in revision 4036 for jbpm 3.2.6 you changed the names of services in org.jbpm.svc.Services from an List to a collection. What's the reason for it?

I think the list is a better choice, since closing the services then occur in the order like in XML, and for some services this can be relevant (e.g. closing the PersistenceService may cause problems if "later" services need something from the database during closing)...

And efficiency isn't a big argument in this 5 services list I think... Not allowing duplicates could be introduced by a simple check in the addService...

Or what do you think?

Cheers
Bernd

[aguizar]

I think you are right, I played with that change a bit to examine its effects, but forgot to revert it when I committed. I'll do it now.

[camunda]

Perfect, thanks!

[tom.baeyens]

ordering of services is crucial ! good that you put it back.