| To: | Martins Innus <minnus@xxxxxxxxxxx>, Ken McDonell <kenj@xxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [pcp] Hotproc fixes |
| From: | Nathan Scott <nathans@xxxxxxxxxx> |
| Date: | Wed, 29 Apr 2015 23:11:55 -0400 (EDT) |
| Cc: | pcp@xxxxxxxxxxx |
| Delivered-to: | pcp@xxxxxxxxxxx |
| In-reply-to: | <554128C1.8080901@xxxxxxxxxxx> |
| References: | <5536A934.8040002@xxxxxxxxxxx> <5537A154.2090803@xxxxxxxxxx> <5537A3F3.5060204@xxxxxxxxxxx> <233739221.5258413.1429764654377.JavaMail.zimbra@xxxxxxxxxx> <55392BAB.3060101@xxxxxxxxxxx> <806884984.5885784.1429839438565.JavaMail.zimbra@xxxxxxxxxx> <554128C1.8080901@xxxxxxxxxxx> |
| Reply-to: | Nathan Scott <nathans@xxxxxxxxxx> |
| Thread-index: | hFpkRc7c8BUXWs0W9qERw/sM+P7DFg== |
| Thread-topic: | Hotproc fixes |
----- Original Message ----- > [...] > This should be all done here: > > https://github.com/ubccr/pcp/tree/hotproc_cleanups Looks good to me. > Also included some other cleanup suggestions from Marko, the ability to > reload the config file, updated qa hopefully for all the changes, merged > the manpage into procpmda, and added a new error code. Feel free to > drop that last one if you think its unnecessary. It should be its own > commit. > I've left that last one for now - needs a bit more discussion I guess, as an ABI extension. I like it though. We should probably audit the three/ four other PMDAs that already use PM_ERR_CONV and switch 'em over. Ken - any other thoughts on a specific error code for bad pmStore to a PMDA? (Oh, maybe the name PM_ERR_BADSTORE to match your error message more closely Martins? the "FMT" part I found a little bit cryptic) cheers. -- Nathan |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | pcp updates: merge nathans (pmdaproc,qa) + Lukas (postfix) + Martins (hotproc), Mark Goodwin |
|---|---|
| Next by Date: | pcp updates: webd, hotproc, docs, Nathan Scott |
| Previous by Thread: | Re: [pcp] Hotproc fixes, Martins Innus |
| Next by Thread: | Re: [pcp] Hotproc fixes, Martins Innus |
| Indexes: | [Date] [Thread] [Top] [All Lists] |