NServiceBus/NServiceBus — comment on commit 44f79a38a54aebfcfe7d3bc070fd90cce523b884
File: src/impl/SagaPersisters/RavenSagaPersister/NServiceBus.SagaPersisters.Raven/RavenSagaPersister.cs line 59
By johannesg:

NH supports multiple attributes. In NH:s case it's really simple, it just
adds a unique constraint for every column marked as unique.

Although I have been thinking of use cases with multiple unique properties,
I don't have a problem supporting only one. On the other hand, I see no
real reason for the core to restrict it just because some saga persisters
cant support it.

How about letting each saga persister implementation decide? Maybe some
implementations simply cant support unique properties. In that case, just
let them throw an exception during startup if someone tries to do it anyway.

2012/1/13 Andreas hlund <
reply@reply.github.com
>

> Johannes: Is the NH saga persister supporting multiple [Unique] attributes?
>
> All: Given that its very rare to need multiple unique properties. Should we
> restrict users to only support one. This makes it easier to implement new
> saga storage's. Users needing more than one should can be guided to set
> their own constraints in manually?
>
> On Fri, Jan 13, 2012 at 2:32 AM, Jonathan Matheus <
> reply@reply.github.com
> > wrote:
>
> > No, I was saying that we can use session.Delete. We can also support
> > multiple properties pretty easily. I can do both tomorrow
> >
> >
> >
> > On Jan 12, 2012, at 3:51 PM, Andreas hlund
> wrote:
> >
> > > So you're saying that we can support multiple unique properties?
> > >
> > > On Thu, Jan 12, 2012 at 6:18 PM, Jonathan Matheus <
> > > reply@reply.github.com
> > >> wrote:
> > >
> > >> We can now that we're using a UOW. We previously opted not to rely on
> > >> Raven change tracking, but recently changed our minds. I'll change it.
> > >>
> > >> ---
> > >> Reply to this email directly or view it on GitHub:
> > >>
> > >>
> >
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-861627
> > >>
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> >
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-862610
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-863209
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-865707
>