[SCM] Samba Shared Repository - branch master updated

[simo]

On Mon, 2010-07-05 at 10:00 +0200, Volker Lendecke wrote:
> On Sun, Jul 04, 2010 at 06:20:29PM -0400, simo wrote:
> > On Sun, 2010-07-04 at 14:49 -0500, Volker Lendecke wrote:
> > > commit 23a31becacee9da11ebe4dff4a3146e19c95a5be
> > > Author: Volker Lendecke <vl at samba.org>
> > > Date:   Sun Jul 4 20:45:43 2010 +0200
> > > 
> > >     s3: Remove unused msg_flags from connections.tdb
> > >     
> > >     This breaks rolling code upgrade!
> > 
> > I guess this is because you removed the msg_flags entry from
> > conenction_data.
> > 
> > Can't we simply rename the field to uint32_t unused and retain
> > compatibility ?
> > 
> > Or would that cause issues I don't see right now ?
> 
> So we are at the point that we can never change internal
> databases anymore?

We do not have any hard rule there.

> This is a new policy that I was not aware of, sorry.

It is not a policy, I was just asking if it made sense to do so.

Allowing for seamless upgrades when possible is a good thing, and we
often added functions to upgrade internal DBs when the DB format
changed.
Is this something we decided is not worthy anymore ?

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>