Merge brlock.tdb and locking.tdb

[Jeremy Allison]

On Sun, Oct 29, 2017 at 08:39:02PM +0100, Volker Lendecke via samba-technical wrote:
> On Sun, Oct 29, 2017 at 06:25:54PM +0100, David Disseldorp via samba-technical wrote:
> > On Sun, 29 Oct 2017 12:29:08 +0100, Ralph Böhme via samba-technical wrote:
> > 
> > > Unless I'm missing something, the record key is the same, we could add the brl data
> > > to the struct share_mode_data IDL definition and update the brlock code to use a
> > > different backing store.
> > > 
> > > This consolidation could be helpful in the implementation of a persistent
> > > handles backing store.
> > > 
> > > Anyone?
> > 
> > Sounds like a good idea to me. 
> 
> One argument for keeping brlock.tdb is the fact that we might have to
> read it MUCH more often to implement mandatory byte range
> locking. You don't want to parse a potentially large and complex
> locking.tdb record for every read or write request.

Yep, that's the issue. That's why the byte range lock
data was originally kept separate from the open file
data.

Jeremy.