Re: [9front] [PATCH 6/6] ndb: document attribute name and value length constraints in BUGS section of man page

[unobe@cpan.org]

Thanks, ori.  I hadn't noticed that!  But I don't see any
corresponding soft limit for the attribute.  Do I understand correctly
that /sys/src/libndb/ndbaux.c:34 just clips the attribute down to 32
bytes?

Quoth ori@eigenstate.org:
> Looking at the code, Ndbvlen seems to be a soft limit; it's only
> the initial allocation to save a malloc, but we can grow past it;
> see ndbsetval in /sys/src/libndb/ndbfree.c:29. There is a soft limit
> coming from Brdline, which depends on the size of the buffer in the
> NDB struct; maybe we should document that a line must be no longer
> than 8k. (This was bumped not so long ago to support the large TXT
> records needed by DKIM).
> 
> That said, I think documenting the attribute size is a good idea.
> 
> 
> Quoth unobe@cpan.org:
> > Ping.
> > 
> > Quoth unobe@cpan.org:
> > > I came across these limitations (defined in /sys/include/ndb.h as
> > > Ndbalen and Ndbvlen, respectively) when toying around with ndb as a
> > > simple database.  This hopefully will save someone else some head
> > > scratching when the query isn't returning as it should.
> > > 
> > > Quoth Romano <unobe@cpan.org>:
> > > > ---
> > > > diff 46cf9d4eb02a3a7c2516be3053994d4fb69eb949 0b5b2d58047b698564f6b1955a31cbcc95570abb
> > > > --- a/sys/man/8/ndb
> > > > +++ b/sys/man/8/ndb
> > > > @@ -924,3 +924,5 @@
> > > >  .I Ndb
> > > >  databases are case-sensitive;
> > > >  ethernet addresses must be in lower-case hexadecimal.
> > > > +.I Ndb
> > > > +database attributes are constrained to 32-byte attribute names and 64-byte attribute values.
> > > > 
> > > 
> > 
>