[00:57] * wgrant finishes another gardening round [00:57] \o/ [00:57] One deployment and 7 further critical fixes until we get back to the first half of last year [00:58] in terms of critical bugs you mean? [00:58] Yeah [06:06] wgrant: have you used storm's ClassAlias? I have an issue where it is spitting out the name of the original table, rather than the alias, when outputting a column. my usage seems to be in line with how it's used elsewhere [06:07] wallyworld_: There's a bug with ClassAliases and References [06:07] wallyworld_: use the _id column instead [06:07] oh, awesome [06:07] thanks [06:07] Storm has bugs? I'm shocked [06:07] eg. ClassAlias(Person).teamowner will compile to Person.teamowner, ClassAlias(Person).teamownerID will be like "_1".teamowner as you'd expect [06:08] lol [06:08] Bug #682989' [06:08] Bug #682989 [06:08] StevenK: How was your vacation? :) [06:08] nigelb: Still on it [06:08] Hah. [06:08] Still in Adelaide, even [06:09] * nigelb really needs to plan an Australian vacation [06:09] wgrant: right, that's the exact issue, thanks. i should have asked earlier. i thought it was me. oh well [06:09] wallyworld_: It's not you, it's me. [06:09] s/me/Storm/ [06:09] yeah [06:10] Flight was 1pm, but has been delayed due to an engineering issue after they had us wait on the tarmac for an hour [06:10] Hopefully we'll board at ~5pm ADL time [06:11] Is that +9.5 or +10.5 atm? [06:11] wgrant: do you know a way to get store.find() to allow a "select 1 from where..." without a table. it's legal postgres sql but storm complains [06:11] +10.5, it's currently 4:40pm [06:12] bah, without the from of course [06:12] wallyworld_: What are you trying to do? [06:12] select 1 where exists(some condition) [06:12] Specifics :) [06:12] only returns one or zero rows [06:13] There's probably a better way [06:13] that's what the tableless select is for [06:13] in oracle it would be "select from dual..." [06:13] If it's literally a SELECT 1 WHERE EXISTS (some subexpression);, just remove the outer expression [06:13] i just want a true/false if some condition exists in the database [06:14] SELECT 1 FROM blah WHERE blah LIMIT 1; [06:14] An exists check without the EXISTS [06:14] i could add limit 1 i guess, but i would be adding a surperfulous table [06:14] No [06:14] I mean with the inner expression [06:14] You don't need the outer one [06:15] What's the statement that you're trying to Stormify? [06:15] this works now: https://pastebin.canonical.com/77001/ [06:16] but i don't need the table in the outer select [06:16] I'm confused [06:16] i could put any table in the using() and it would work [06:17] Isn't that just "return not Store.of(self).find(SourcePackageRelease, SourcePackagePublishingHistory.sourcepackagereleaseID == SourcePackageRelease.id, Archive.id == SourcePackagePublishingHistory.archiveID, *clauses).is_empty()"? [06:17] probably, i didn't know about is_empty [06:18] is that efficient? [06:18] It's shorthand for SELECT 1 blah blah blah LIMIT 1 [06:18] Not when you're linking SPPH and SPR [06:18] it will be a whole lot better than that is done now luckily [06:18] Even if it didn't exist, I'd just manually do a SELECT 1 FROM spr JOIN spph blah blah blah LIMIT 1 [06:19] No need for a subquery [06:19] i wasn't sure how ggod postgres would be at that [06:19] i assume it will be smart about seeing the limit 1, and stop computing as soon as it finds a matching row [06:23] Right [06:23] Just as it doesn't compute the entire result of Ubuntu's bugs just to show the first 75 of them [06:26] So, who's the new product manager for LP? [06:26] Nobody yet [06:27] And no new lifeless either? [06:27] Also not yet [06:27] :) [07:06] wallyworld_: btw [07:06] In [14]: list(store.execute(Select(1, where=Exists(Select(SourcePackageRelease.id, tables=[SourcePackageRelease], where=True))))) [07:06] Out[14]: [(1,)] [07:06] In [15]: list(store.execute(Select(1, where=Exists(Select(SourcePackageRelease.id, tables=[SourcePackageRelease], where=False))))) [07:06] Out[15]: [] [07:06] It works fine without tables [07:07] Not sure what you were running into [07:25] good morning === gmb_ is now known as gmb === gmb_ is now known as gmb === almaisan-away is now known as al-maisan [08:43] morning [08:44] anyone know which team worked on the hide a commetn feature on LP bugs ? [08:44] *comment [08:45] bzr annotate is your friend [08:45] bigjools: morning [08:46] g'day :) [08:46] bigjools: you at the hotel ? [08:46] sprinting, yup [09:00] czajkowski: That was finished off by Purple near the end of one of the subfeatures [09:00] I'm not really happy with how it turned out, but such is life [09:00] wgrant: was just curious as there is a bug/feature issue being reported on it [09:00] just wondered if you knew more about it [10:15] Could I have a DB patch number for work on bug 1068071, please? [10:15] <_mup_> Bug #1068071: Need facility to redirect Ubuntu uploads to non-release pocket < https://launchpad.net/bugs/1068071 > [10:17] cjwatson: you need jtv or wgrant I think as stub is away this week [10:18] czajkowski: Any LP developer can claim a patch number. [10:18] That's why I just asked generally. [10:18] You only have to be in ~launchpad. [10:18] oh ok. [10:21] cjwatson: Sure [10:28] cjwatson: 2209-36-0 is yours [10:28] Thanks === al-maisan is now known as almaisan-away === almaisan-away is now known as al-maisan === gmb` is now known as gmb === al-maisan is now known as almaisan-away [13:39] abentley: https://code.launchpad.net/~adeuring/launchpad/sec-adapter-projectgroup-milestone/+merge/130800 [13:45] adeuring: Did you consider using DelegatedAuthorization? [13:45] abentley: not, i was not aware of this [13:46] adeuring: I think it simplifies things a bit. [13:46] right [13:53] abentley: changed [13:54] adeuring: I think milestone.userCanView is dead code now. [13:55] abentley: gahh, forgot to save that file before commtting... [13:56] done now [13:56] adeuring: Cool. [13:59] adeuring: You're indirectly asserting what the contents of "IProjectGroupMilestone" should be. I'm okay landing this way, but let's talk about this in Copenhagen. I bet we can come up with an equivalent that takes IProjectGroupMilestone as its input. Like verifyObject. [13:59] abentley: sure [13:59] adeuring: r=me. [13:59] thanks [14:00] adeuring: np. [14:02] jcsackett, do you have time to talk? [14:08] sinzui: sure, one moment while i find my phone. [14:29] https://bugs.launchpad.net/launchpad/+bug/164530 [14:29] <_mup_> Bug #164530: Translation import queue showing broken links < https://launchpad.net/bugs/164530 > [14:36] https://bugs.launchpad.net/launchpad/+bug/227380 [14:36] <_mup_> Bug #227380: SourceForge ExternalBugTracker doesn't handle non-existent bugs < https://launchpad.net/bugs/227380 > [14:37] jcsackett, https://bugs.launchpad.net/launchpad/+bug/227380 [14:37] <_mup_> Bug #227380: SourceForge ExternalBugTracker doesn't handle non-existent bugs < https://launchpad.net/bugs/227380 > [14:42] adeuring: Apparently, DelegatedAuthorization is also performance improvement, because it allows LaunchpadSecurityPolicy to cache Product security checks. [14:43] abentley: right [14:45] jcsackett, https://bugs.launchpad.net/launchpad/+bug/924292 [14:45] <_mup_> Bug #924292: TypeError raised during email authentication when the email contains non-ascii characters < https://launchpad.net/bugs/924292 > === matsubara is now known as matsubara-lunch [16:08] for IBranchMergeProposal.createComment, 'subject' is mandatory. How do I specify "just use the default subject that would be used if I were commenting through the web UI"? [16:09] * jml experiments [16:12] jml: the guard for that in the model code is "if not subject:", so subject="" should work for that [16:12] cjwatson: thanks. subject=False seems to work too. [16:13] abentley, adeuring, rick_h_ -- hi, guys. I reviewed bugs for the product relationship, and as near as I can tell, there's only one issue. [16:13] lib/lp/app/javascript/comment.js uses subject: '' [16:13] cjwatson: Interesting work on archive for raring. [16:13] deryck: cool [16:13] abentley, adeuring, rick_h_ -- and I filed a bug and added a card to the board, but it's not beta critical. [16:13] cjwatson: Is this the LP work you were doing this cycle? [16:13] deryck: Great. [16:14] abentley, adeuring, rick_h_ -- it's a bit of a weird situation to reproduce, so we can get it post beta. [16:14] deryck: ok [16:14] nigelb: There's only quite a small LP component to this, since the copy system is now fairly flexible. Just a few bug fixes and tweaks. [16:15] cjwatson: Ah. [16:15] nigelb: I was intending to finish off the replace-archive-admin-shell-access work if I could. [16:16] (Which is basically chroot management and copy archives.) [16:16] Nothing hugely exciting. [16:17] That means AA work will not need shell access? [16:18] Hah, no. [16:18] False does not work [16:18] cjwatson: ta. will try that. [16:19] nigelb: It mostly no longer does. [16:19] Only a very small number of rare operations. [16:20] Aha [16:20] Oh, UDS is european this time. I should try to listen to a few sessions at least. === matsubara-lunch is now known as matsubara === beuno is now known as beuno-lunch [17:11] sinzui: did you get the Rt I created to track the translation import issue that was logged via Answers ? [17:12] No, Do I need once since I need to track it on my kanban board and I have that card [17:12] already assigned [17:13] sinzui: well was trying out this new policy where I can track an issue that I raise with maintenance [17:14] that way instead of poking for an update I can check the RT [17:14] like all the other depts do in canonical [17:14] We will update it when we do something next [17:14] thank you [17:14] now to go take the 3 year old for a walk who has been sitting patiently with her new kite [17:14] toodles [17:15] DatabaseSchemaChangesProcess still has some stuff about slony and the todrop namespace, which seems to have been removed in devel r15757. Should that be removed from the process? === almaisan-away is now known as al-maisan === al-maisan is now known as almaisan-away === beuno-lunch is now known as beuno === Ursinha-afk is now known as Ursinha === Ursinha-afk is now known as Ursinha [22:54] wallyworld_, this is the list of related issues https://bugs.launchpad.net/launchpad/+bugs?field.tag=related-projects-packages