#general
- w
witty-crayon-22786
10/22/2015, 7:40 PMyea. can just do that first and think about the rest of the implications later. - w
witty-crayon-22786
10/22/2015, 7:41 PM@enough-analyst-54434: cc ^, in case it's somehow relevant to the engine - e
enough-analyst-54434
10/22/2015, 7:43 PMRight now the engine stuff has no care about invalidation - not part of the model. The experiments are only focused on "can we schedule this?" - w
witty-crayon-22786
10/22/2015, 7:43 PMok, cool beans. so the complete graph is in memory, and then gets pruned afterward. - w
witty-crayon-22786
10/22/2015, 7:43 PMmakes sense, thanks. - e
enough-analyst-54434
10/22/2015, 7:43 PMIt could get pruned either in the plan stage or execution stage - e
enough-analyst-54434
10/22/2015, 7:43 PMThat' - e
enough-analyst-54434
10/22/2015, 7:43 PMs left open - w
witty-crayon-22786
10/22/2015, 7:44 PMtrue. - w
witty-crayon-22786
10/22/2015, 7:44 PMthanks - e
enough-analyst-54434
10/22/2015, 7:46 PMI'll be working up a README for the engine stuff one of these nights before I fly to SF, hopefully tonight, maybe Friday night - for homework. I'd like it if folks had time to understand what's going on before the meeting - w
witty-crayon-22786
10/22/2015, 7:46 PMyes please. - b
bored-art-40741
10/22/2015, 7:51 PMA quick list of the relevant CRs to read for those behind would also be handy - w
witty-crayon-22786
10/22/2015, 7:58 PMstill a bit unclear... are python mixins that override methods sane? - e
enough-analyst-54434
10/22/2015, 7:58 PMThis should work: https://github.com/pantsbuild/pants/pulls?q=label%3Aengine+type%3Apr - b
bored-art-40741
10/22/2015, 7:59 PM@witty-crayon-22786: We've gotten more comfortable with using them in limited circumstances - b
bored-art-40741
10/22/2015, 7:59 PMThe two important rules are: - b
bored-art-40741
10/22/2015, 7:59 PM(1) ALWAYS call super - w
witty-crayon-22786
10/22/2015, 7:59 PMyep. - b
bored-art-40741
10/22/2015, 7:59 PM(2) ALWAYS keep mixins on the left - w
witty-crayon-22786
10/22/2015, 8:00 PMok, cool. in particular, i want to override
to add more from a mixin.register_options - b
bored-art-40741
10/22/2015, 8:00 PMMixins overriding other mixins is probably crazytown though - b
bored-art-40741
10/22/2015, 8:00 PMAh, I think that's probably a good use - w
witty-crayon-22786
10/22/2015, 8:00 PMk - b
bored-art-40741
10/22/2015, 8:00 PMWhere you're just inserting a link into a chain - w
witty-crayon-22786
10/22/2015, 8:00 PMyep. - b
bored-art-40741
10/22/2015, 8:00 PMAll you care about is that the MRO is linearized sanely and the order in which you register your options doesn't matter - w
witty-crayon-22786
10/22/2015, 8:00 PMusually overriding concrete is insane though... this is basically the only case where i feel comfortable with overrides. - w
witty-crayon-22786
10/22/2015, 8:00 PMyea - w
witty-crayon-22786
10/22/2015, 8:00 PMk, thanks.