BIND 10 trac1207, created. d45f8ba1689dbdce4e6a3f36fca128f13d4079a3 [1208] Remove InMemoryClient-specific code from auth_srv

BIND 10 source code commits bind10-changes at lists.isc.org
Tue May 8 10:18:52 UTC 2012


The branch, trac1207 has been created
        at  d45f8ba1689dbdce4e6a3f36fca128f13d4079a3 (commit)

- Log -----------------------------------------------------------------
commit d45f8ba1689dbdce4e6a3f36fca128f13d4079a3
Author: Jelte Jansen <jelte at isc.org>
Date:   Tue May 8 11:53:24 2012 +0200

    [1208] Remove InMemoryClient-specific code from auth_srv
    
    Made the getZoneCount() method from inmemory client a general datasource client
    call (which throws NotImplemented by default), as this does seem useful in other
     cases as well.
    
    Now the work starts to pay off; removed the temporary pointer variable (getInMemoryClientP() still returns a pointer but now directly retrieved from the container), and any direct reference to isc::datasrc::InMemoryClient is removed from au
    th_srv (but not from tests as of yet)

commit 0b5f0023e7cde8924891f75686e93c0c5fead7ad
Author: Jelte Jansen <jelte at isc.org>
Date:   Tue May 8 10:46:50 2012 +0200

    [1207] remove old-style inmem-ds getter and setter

commit bf1717f38c4292fcd19d31ba69f810079f80664d
Author: Jelte Jansen <jelte at isc.org>
Date:   Mon May 7 15:19:19 2012 +0200

    [1208] use the container to set in-mem client
    
    Instead of the direct shared_ptr.
    
    Changes needed:
    Moved loading of zones in config into the memory_datasrc implementation; at this moment the general API has no method for this (and it may not be necessary; datasources should handle their own config anyway).
    This does have one drawback; through the dynamic loading mechanism we currently lose exception type information. So for now specific exceptions such as MasterLoadError end up as 'general' DataSourceErrors. Tests have been updated to reflect this.
    
    Also, I made config handler slightly more lenient; it has hardcoded defaults and will now accept config parts without "class" or "zones"; using .spec -derived values is impractical here with the current config API, but in this case it shouldn't *really* matter; as IN is hardcoded in several places anyway atm.

commit 18cb352fef0dc9343fbffad36539bb7f1cce5214
Author: Jelte Jansen <jelte at isc.org>
Date:   Mon May 7 12:33:37 2012 +0200

    [1207] add setInMemoryClient with container instead of shared ptr

commit 446e309a0456d8fbf3c53a66d7cc0f556bda9b6b
Author: Jelte Jansen <jelte at isc.org>
Date:   Mon May 7 12:25:28 2012 +0200

    [1207] store and use raw inmem client pointer
    
    (temporarily)
    In order to add datasource factory, but keep current code working, store both the shared ptr (for reference counting) and the raw pointer it encapsulates. The latter is then used to call query::process().

commit 9a1f442d5067ecc75eecf2678cc90d74d4a5e597
Author: Jelte Jansen <jelte at isc.org>
Date:   Mon May 7 11:22:22 2012 +0200

    [1207] add shared_ptr DataSourceClientContainerPtr

-----------------------------------------------------------------------



More information about the bind10-changes mailing list