Home
openafs@conference.openafs.org
Friday, September 30, 2016< ^ >
Room Configuration
Room Occupants

GMT+0
[01:58:06] mvita leaves the room
[01:58:07] mvita joins the room
[02:06:47] mvita leaves the room
[02:08:19] mvita joins the room
[02:13:42] mvita leaves the room
[03:16:25] mvita joins the room
[04:15:50] mvita leaves the room
[07:06:26] Simon Wilkinson joins the room
[11:56:56] Marcio Barbosa joins the room
[13:25:16] Jeffrey Altman leaves the room
[13:25:17] Jeffrey Altman joins the room
[14:58:38] mvita joins the room
[15:41:09] <kadukoafs@gmail.com/barnowl9C09F4AA> mvita: bold move (on 12389)
[15:43:07] <mvita> ?
[15:43:40] <kadukoafs@gmail.com/barnowl9C09F4AA> I put a -1 review on it
[15:44:00] <mvita> oh, sorry, somehow I missed that..
[15:44:33] <mvita> looking at the other commit.
[15:45:53] <mvita> oh that is so funny!  I almost put in my review "we missed an opportunity to do some common code consolidation here" or words to that effect
[15:46:22] <mvita> and I added the refactor to my back burner list
[15:48:01] <mvita> oh, Marc's is much nicer than what I had in mind.
[15:48:33] <kadukoafs@gmail.com/barnowl9C09F4AA> I didn't see how clean the cherry-pick was, though.
[15:52:15] <mvita> > his change only affects one of several call sites.
[15:52:23] <mvita> s/his/this
[15:53:04] <mvita> I'm not seeing any other ContactQuorum* routines that were missed - I think the current patchset does cover them all, even without Marc's refactor
[15:53:21] <mvita> what were you referring to?
[15:53:53] <kadukoafs@gmail.com/barnowl9C09F4AA> Hmm, maybe I just failed to scroll.  Whoops.
[15:54:35] <mvita> I know because I was working out in my head where the common code was in all of them
[15:54:36] meffie joins the room
[15:54:55] <kadukoafs@gmail.com/barnowl9C09F4AA> *nods*
[16:02:48] <mvita> re: d_prune_alias - there is a very good discussion in the review comments for https://gerrit.openafs.org/#/c/8751
[17:09:30] ballbery leaves the room
[17:09:31] ballbery joins the room
[17:17:37] meffie leaves the room
[21:15:43] Marcio Barbosa leaves the room
Powered by ejabberd Powered by Erlang Valid XHTML 1.0 Transitional Valid CSS!