Particular / NServiceBus

Build, version, and monitor better microservices with the most powerful service platform for .NET
https://particular.net/nservicebus/
Other
2.09k stars 648 forks source link

Rename UniquelyIdentifyRunningInstance #4540

Open timbussmann opened 7 years ago

timbussmann commented 7 years ago

I think the API is misleading and hard to really understand. Also it kinda conflicts with MakeInstanceUniquelyAddressable. Also it's weird to use as UniquelyIdentifyRunningInstance takes no parameter but returns a configuration class which contains different properties where

During the acceptance test reviews we concluded that the host headers are essentially diagnostics data. I think this would be a much more suitable API category, e.g. something like:

endpointConfiguration.Diagnostics(settings => 
{
    settings.CustomDisplayName = "whatever";
    settings.HostId = HostId.Guid(Guid.NewGuid()); // HostId.FromString(endpoint, instance, ...);
});
andreasohlund commented 7 years ago

@timbussmann since we hopefully will move in the direction of a explicit host config api should we consider closing this?

andreasohlund commented 7 years ago

Bumping to the Future milestone for now, since https://github.com/Particular/NServiceBus/pull/4952 will likely make this a non issue so no need to churn the API in v7