BIND 10 #2207: define and implement (datasrc::memory::)ZoneUpdater class
BIND 10 Development
do-not-reply at isc.org
Fri Oct 19 20:41:54 UTC 2012
#2207: define and implement (datasrc::memory::)ZoneUpdater class
-------------------------------------+-------------------------------------
Reporter: | Owner: jinmei
jinmei | Status: reviewing
Type: task | Milestone:
Priority: | Sprint-20121023
medium | Resolution:
Component: data | Sensitive: 0
source | Sub-Project: DNS
Keywords: | Estimated Difficulty: 7
Defect Severity: N/A | Total Hours: 0
Feature Depending on Ticket: |
background zone loading |
Add Hours to Ticket: 0 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
Replying to [comment:16 vorner]:
The branch looks okay for merge (note: I made one final small cleanup).
One non-blocking comment: if you mean by
{{{#!cpp
// For some reason it doesn't seem to work by the ZoneNode typedef,
using
// the full definition instead.
}}}
that this doesn't compile for you:
{{{#!cpp
const ZoneNode* node;
EXPECT_EQ(ZoneTree::EXACTMATCH,
}}}
it worked here. So, that's probably compiler dependent.
> > > In that case, there's a confusion about `Unexpected`. Grep through
the `memory/` directory for it and you'll see many occurrences of
`Unexpected`, and none of them looks like a system error.
> >
> > I guess many of you were added by yourself:-) but in any case I admit
> > it hasn't been officially clarified which exception (especially
> > `Unexpected`) should be when. We'll probably need a cleanup (and
> > documentation) ticket.
>
> I'm not aware of me adding them, but I might have short memory. Will you
create the clean-up ticket, please?
Okay.
--
Ticket URL: <http://bind10.isc.org/ticket/2207#comment:17>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list