On Sat, 2008-12-06 at 10:48 -0500, Raymond Dans wrote:
> Scott wrote:
> 
> ... Snip
> 
> >> 3. When the SipxSupervisor Process Manager asks to verify an IMDB 
> >> resource, the following should occur:
> >> 
> >>    Check to see if the table exists:  
> >> 
> >>    If it does NOT exist yet, see if there is an xml file for this 
> >> table and if so load it.  If no xml file exists then the resource is 
> >> NOT satisfied and will return indicating this.
> >
> >that's correct (verify that this is the behavior of the 
> >getInstance through the SIPDBManager for each)
> >
> Unfortunately by calling getInstance, a loading of the table will be
> attempted using the xml file.  Unfortunately, even though an xml file
> may not be found, no error status is returned (always returns success).
> A database pointer is always returned if the database (nothing to do
> with the table) was successfully opened.  At least that's how it looks
> to me.
> The code is somewhat confusing to look at because the word Database
> seems to be used interchangeably with tablename and in a database world,
> these are not the same thing.  As you know, a Database contains
> potentially many tables but each table is not a database per se.
> 
> I think we could modify each individual table such that if it fails to
> find an xml file for it, it will nil out the DB pointer and then I think
> the rest will fall into place.  Does this sound reasonable?

Yes, very good.

I recently added improved error reporting for reading/parsing xml files
- see:

 sipXportLib/include/xmlparser/XmlErrorMsg.h


_______________________________________________
sipx-dev mailing list
[email protected]
List Archive: http://list.sipfoundry.org/archive/sipx-dev
Unsubscribe: http://list.sipfoundry.org/mailman/listinfo/sipx-dev

Reply via email to