Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

Commit

Permalink
Added a better comment regarding the selection of Persistence,Fixes #27
Browse files Browse the repository at this point in the history
… , Also removed the AsA_Server as its no longer needed. Fixes #1
  • Loading branch information
indualagarsamy committed Oct 24, 2014
1 parent dadea56 commit 3af780c
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion packaging/nuget/nservicebus.host/EndpointConfig.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,15 @@ namespace rootnamespace
This class configures this endpoint as a Server. More information about how to configure the NServiceBus host
can be found here: http://particular.net/articles/the-nservicebus-host
*/
public class EndpointConfig : IConfigureThisEndpoint, AsA_Server
public class EndpointConfig : IConfigureThisEndpoint
{
public void Customize(BusConfiguration configuration)
{
// For production use, please select a durable persistence.
// To use RavenDB, install-package NServiceBus.RavenDB and then use configuration.UsePersistence<RavenDBPersistence>();
// To use SQLServer, install-package NServiceBus.NHibernate and then use configuration.UsePersistence<NHibernatePersistence>();

// For development/debugging you can also use, configuration.UsePersistence<InMemoryPersistence>();

This comment has been minimized.

Copy link
@johnsimons

johnsimons Nov 17, 2014

Member

@andreasohlund I thought InMemoryPersistence is ok for prod use too? Specifically when transactions are off.

This comment has been minimized.

Copy link
@andreasohlund

andreasohlund Nov 17, 2014

Member

true, we should update the text to instead warn that its not a durable store so it should be used with care.

Will you update the text?

This comment has been minimized.

Copy link
@johnsimons

johnsimons Nov 17, 2014

Member

You do, I'm not sure exactly how to phrase it, can u help?

This comment has been minimized.

Copy link
@andreasohlund

andreasohlund via email Nov 17, 2014

Member

This comment has been minimized.

Copy link
@johnsimons

johnsimons Nov 17, 2014

Member

Thanks, let me know when u done, and I'll proceed with the release

configuration.UsePersistence<PLEASE_SELECT_ONE>();
}
}
Expand Down

0 comments on commit 3af780c

Please sign in to comment.