search

tdiesler / fabric8poc

POC for the new Fabric8 API
Apache License 2.0
2 stars 5 forks source link

ServiceEndpoint now uses a String identity. Added String type on Service... #29

Closed iocanel closed 10 years ago

iocanel commented 10 years ago

...Endpoint. Added ServiceEndpointFactory. Added ServiceLoader for finding the appropriate ServiceEndpointFactory for each Service EndpointType.

tdiesler commented 10 years ago

I'd like to preserve ServiceEndpointIdentity. It should extend AbstractIdentity, which does a validation on the symbolic name. In fact all identities should now implement Identity, which defines the valid id pattern.

We may also want to introduce ProfileItemIdentity to uniformly have the same id guarantees. Anyway, an arbitrary string is not a good identity.