* danilos -> lunch | 10:49 | |
gary_poster | bac benji danilos (gmb sick :-( ): call in 2 | 12:29 |
---|---|---|
benji | g*ry: RT is 48196 | 12:38 |
danilos | bac is fighting irc problems, he'll be unavailable on IRC while that's going on | 12:48 |
gary_poster | danilos, thanks for update on bac. I need to take kids to school today, so I will be a bit late. Will 20 minutes past the hour still work for you, or should we reschedule? | 12:50 |
danilos | gary_poster, that's fine | 12:57 |
gary_poster | thanks danilos. ready any time | 13:18 |
danilos | gary_poster, hey, still not used to non-existent notifications :) | 13:24 |
gary_poster | heh | 13:24 |
gary_poster | benji, when would you like a call this morning? Any time is good for me. | 13:57 |
benji | gary_poster: sure; how about now? | 13:58 |
gary_poster | benji, a fine idea. Gimme a call! | 13:58 |
bac | hi danilos | 14:02 |
bac | danilos: the original failing test is bugs/doc/bugtask.txt at line 335. the scenario presented there is the one i've tried to recreate in the unit test. | 14:03 |
danilos | bac, right, let me take a look at as well | 14:12 |
danilos | bac, hum, right, I think I see the point now | 14:25 |
bac | ok | 14:25 |
bac | danilos: my understanding of conjoinedness is not deep, so i don't deny that i may have gotten them backwards but the doctest regression looks real | 14:26 |
bac | btw, anyone have an idea how long fsck should take on a 1TB disk? | 14:28 |
danilos | bac, yeah, it all seems right, in this special case when a master is being created after slave, it copies stuff from slave back to master | 14:34 |
danilos | bac, so, I think the problem is the storm validator: createTask creates a task with status set to NEW, which calls on the storm validator for status field, which changes the slave version back to the just-created master version | 14:36 |
bac | danilos: yes, i was looking at that...before my server died | 14:37 |
bac | danilos: but none of that code has changed... | 14:39 |
danilos | bac, well, at least the column is being renamed from status to _status :) | 14:43 |
bac | danilos: yeah, there is that... | 14:44 |
danilos | bac, fwiw, I can also see arbitrary SQL queries using ._status and others using .status which kind of confuses me | 14:48 |
danilos | that could be storm vs sqlobject stuff, but just very confusing | 14:49 |
=== bac`` is now known as bac | ||
danilos | bac, so, I could confirm that the NEW status is set by the validator, and what's else validate_status is called 9 times now, whereas stable did it 7 times for the same test (your earlier one) | 15:12 |
danilos | bac, fwiw, doctest does seem to break with all those _status usages in plain-text SQL queries (as I expected) when validator is removed, so it seems the branch was more unfinished then you hoped for :) | 15:13 |
bac | danilos: yeah, that's my fear. fix this blocking problem only to find a ton more | 15:14 |
danilos | bac, ok, so the problem is that on stable, self.bug in validator is not set, yet in this branch it is | 15:41 |
danilos | bac, basically, it seems that bug gets flushed earlier with all the changes | 15:43 |
benji | I'm going away now. I suspect I'll keep meditating upon the visions of Python regexes which parse Perl regexes that are swirling around in my head though. | 16:10 |
bac | danilos: ah, interesting | 16:26 |
danilos | bac, I actually dislike the extension to enum columns lifeless did, I find code that mixes two different DBEnums for a single value very confusing :/ | 16:44 |
bac | danilos: how would you do it? | 16:44 |
danilos | bac, usual class inheritance, like we do it everywhere else | 16:45 |
danilos | (we can't have one class that derives from two DBEnumeratedTypes though, but we can have non-dbenum classes as base combining classes) | 16:47 |
gary_poster | lunch | 17:12 |
* danilos -> off, tty tomorrow | 17:42 | |
bac | hi gary_poster, i think (with danilos' help) i now understand what is going on...but i need to discuss it | 18:26 |
gary_poster | bac, ok, cool. now is fine, or later | 18:27 |
bac | gary_poster: now would be good | 18:27 |
gary_poster | we can segue into our weekly call if you want | 18:27 |
gary_poster | cool | 18:27 |
gary_poster | call me when ready bac | 18:28 |
bac | gary i'll resume our call in a sec | 18:49 |
gary_poster | bac ok, I'm back | 18:50 |
bac | me too | 18:54 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!