On Fri, Dec 22, 2006 at 05:00:51PM -0800, Danek Duvall wrote:
> On Fri, Dec 22, 2006 at 02:41:50PM -0800, Stephen Lau wrote:
> 
> > via __init__.py, the Checks package now has a cheesy onSWAN() method
> > that simply does a DNS lookup of sunweb.central to determine if you are
> > on or off-SWAN.
> 
> I've found that (at least in some environments) having the terminating
> period makes a difference in performance -- the negative result returns
> more quickly.

terminating period?

> > DbLookups.py contains classes to wrap bug queries (via Monaco), and ARC
> > queries (via sac.eng).
> 
> The monaco query should be restructured -- what's in wx right now isn't
> very robust to the pipe-table snafus we've been having from time to time.
> You should replace "do query/CR" with something that doesn't use the pipe
> table, namely "doMeta genQuery cr".  However, since you're pulling a lot
> more than just the bugid and the synopsis, it may be that you need the pipe
> table.  You don't seem to be using anything more than just the bugid and
> synopsis, so for now you might be able to back down from the fuller query.
> 
> Just a thought, as I have seen problems with this in the past.

yeah, i wasn't sure if this might be useful for more general things down
the line, so i thought making a generic Monaco object to query and grab
all fields could be useful.  at the moment, i only use the synopsis.

could you go into more detail about this 'pipe-table snafu'?

cheers,
steve
-- 
stephen lau // [EMAIL PROTECTED] | 650.786.0845 | http://whacked.net
opensolaris // solaris kernel development
_______________________________________________
tools-discuss mailing list
[email protected]

Reply via email to