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

Yes, lets go with that!

On Fri, Jan 13, 2012 at 11:22 PM, Johannes Gustafsson <
reply@reply.github.com
> wrote:

> 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
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-866345
>