/srv/irclogs.ubuntu.com/2015/01/16/#launchpad-dev.txt

=== 18VABY2YZ is now known as elmo
=== seelaman` is now known as seelaman
=== beuno_ is now known as beuno
=== Nigel_ is now known as G
=== Spads_ is now known as Spads
=== lan3y is now known as Laney
=== rvba` is now known as rvba
=== jelmer_ is now known as jelmer
=== jelmer is now known as jelmer_
rpadovanilaunchpad on duckduckgo :D16:53
rpadovanihttps://duckduckgo.com/?q=lp+%231&ia=launchpad16:53
cjwatsonrpadovani: says "More at Launchpad.com" which is a domainsquatter and not us.17:03
cjwatsonbut good to see explicit support17:03
rpadovanicjwatson, ops, I reported it in the MR, they didn't fix it :/ I report it to the developer17:05
rpadovani(again)17:05
cjwatsonThe link goes to the right place, it's just the text17:06
=== mbarnett` is now known as mbarnett
cjwatsonwgrant: Do you happen to remember why accessartifact_maintain_denorm_to_artifacts_trig is redefined in database/schema/patch-2209-28-6.sql, apparently identically to its definition in database/schema/patch-2209-16-6.sql?  Is it because the PL/pgSQL function needs to be recompiled to refer to the new definition of accessartifact_denorm_to_artifacts?18:02
cjwatson(I'm guessing so, but wanted to check)18:02
rpadovanifixed launchpad.com on ddg18:04
rpadovanihttps://github.com/duckduckgo/zeroclickinfo-spice/pull/1423/files18:04
cjwatsonthanks!18:08
wgrantcjwatson: Seems pretty pointless to me.22:52
cjwatsonwgrant: OK, so PL/pgSQL PERFORM looks up the other function by name rather than having a precompiled handle to it or something?23:09
wgrantOh right, I remember the history here.23:11
wgrantI can't see why the function was redefined, though. Might be worth deleting it from the patch and running the spec privacy tests.23:13
wgranthttps://bugs.launchpad.net/launchpad/+bug/1067616 ftr23:13
mupBug #1067616: Blueprints could use a denormalized schema for improved performance <private-projects> <qa-ok> <regression> <specifications> <tech-debt> <Launchpad itself:Fix Released by stevenk> <https://launchpad.net/bugs/1067616>23:13
wgrantI don't think I reviewed any of this series because reasons.23:13
wgrantOh, I did.23:13
wgrantJust not the DB patch.23:13
cjwatsonYeah, I'd found that bug.23:15
cjwatsonOK, will experiment if you don't know off the top of your head.23:15
wgrantcjwatson: lp.blueprints tests still pass with that removed, as expected.23:21
cjwatsonOh good.  Thanks.23:22
wgrantI would have been rather scared if they didn't.23:22

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!