| 00:28.53 | *** join/#storm fcorrea (~fcorrea@201-62-110-81-ma.static.vivax.com.br) |
| 01:27.48 | *** join/#storm wallyworld_ (~quassel@27-33-46-253.static.tpgi.com.au) |
| 05:04.04 | *** join/#storm stub (~stub@125.24.103.40.adsl.dynamic.totbb.net) |
| 05:04.09 | *** join/#storm stub (~stub@canonical/launchpad/stub) |
| 05:30.29 | *** join/#storm m4v (~znc@unaffiliated/m4v) |
| 07:27.32 | *** join/#storm stub (~stub@125.24.103.40.adsl.dynamic.totbb.net) |
| 07:27.33 | *** join/#storm stub (~stub@canonical/launchpad/stub) |
| 08:06.36 | *** join/#storm sidnei (~sidnei@201-11-209-227.cslce701.dsl.brasiltelecom.net.br) |
| 08:07.54 | *** join/#storm zmijunkie (~sascha@pd95b6fd9.dip0.t-ipconnect.de) |
| 08:45.07 | _mup_ | storm/documentation r386 committed by jkakar@kakar.ca |
| 08:45.07 | _mup_ | - Fixed incorrect property/Python/database type mappings. |
| 08:45.46 | lifeless | jkakar: my with_ branch seems stalled. |
| 08:45.59 | lifeless | jkakar: gustavo has said 'not good enough', but not said what will be good enough. |
| 08:46.07 | jkakar | lifeless: Ah, I see. |
| 08:46.33 | jkakar | lifeless: I'll take a look at it in the next day or so and see if I can provide some momentum. :) |
| 08:47.07 | jkakar | lifeless: On that topic, if you feel like some simple reviews I have json-property and uuid-bug-fixes branches in review, both of which would be nice to land. :) |
| 08:47.51 | lifeless | jkakar: in storm? I'm not in the reviewers team, so I don't know how much my opinion would matter... |
| 08:48.16 | jamesh | lifeless: I think his idea is that you'd be able to create an object that acted similar to a normal table class but was backed by a Select() |
| 08:48.21 | jkakar | lifeless: In Storm, yes. You're opinion matters a lot, you don't need to be on the Storm team. |
| 08:48.28 | jkakar | Your, even. |
| 08:48.38 | jamesh | and then have the expression compiler automatically generate the WITH clause |
| 08:49.10 | jkakar | jamesh: Something like result = store.find(With(MyObject), ...)...? |
| 08:49.26 | jkakar | needs to learn more about WITH |
| 08:49.53 | jamesh | jkakar: well, you'd want a column list so maybe via a get_select() style mechanism |
| 08:50.08 | lifeless | I've commented on the review. |
| 08:50.09 | jkakar | jamesh: Ah, okay. |
| 08:50.35 | jamesh | jkakar: basically it lets you evaluate a query and use it as a temporary table for the execution of a second query |
| 08:50.53 | lifeless | jamesh: yes, I think that is roughly what he meant, but its out of my immediate reach for doing. |
| 08:51.19 | lifeless | jamesh: we have, since I put this branch up, solved 5? 6? performance bugs with it; fugly or not its working :) |
| 08:51.52 | lifeless | jkakar: jamesh: it also lets you implement recursion (strictly speaking iterative processing) in a single query. |
| 08:52.19 | jamesh | lifeless: I don't doubt that it can do that. |
| 08:52.20 | *** join/#storm danilos (~danilo@cable-178-148-38-107.dynamic.sbb.rs) |
| 08:52.20 | *** join/#storm danilos (~danilo@canonical/launchpad/danilos) |
| 08:55.22 | jamesh | lifeless: I guess one of niemeyer's concerns is that it is pretty closely tied to PostgreSQL, while the alias option leaves the door open to rewrite as a subquery for other databases. |
| 08:56.54 | lifeless | jamesh: abstractions leak |
| 08:57.21 | jamesh | or perhaps he just thinks the alias-like approach would result in nicer code |
| 08:57.34 | lifeless | jamesh: by which I mean: this is not something you trivially rewrite; WITH is a sql standard; the correct rewrite will require actual temp tables |
| 08:57.45 | lifeless | jamesh: and rewriting recursive with will be... interesting |
| 08:59.26 | lifeless | jamesh: it may; if someone guides me - and some heavy guidance will be needed I suspect - I am happy to invest moderately more time into this. |
| 09:00.07 | jamesh | lifeless: I'll add a comment to the merge proposal. I'm not entirely sure what the API should look like, but will try to help. |
| 09:00.51 | lifeless | cool |
| 09:04.25 | *** join/#storm bigjools (~quassel@82-71-93-254.dsl.in-addr.zen.co.uk) |
| 09:04.25 | *** join/#storm bigjools (~quassel@canonical/launchpad/bigjools) |
| 09:13.37 | *** join/#storm zmijunkie (~sascha@pd95b6fd9.dip0.t-ipconnect.de) |
| 09:15.37 | jkakar | therve: Thanks for the reviews! :) |
| 09:16.02 | therve | jkakar, you're welcome |
| 09:27.04 | *** join/#storm jkakar (~jkakar@127.Red-83-60-96.dynamicIP.rima-tde.net) |
| 13:01.02 | *** join/#storm niemeyer (~niemeyer@201-66-179-18.pltce701.dsl.brasiltelecom.net.br) |
| 14:34.55 | *** join/#storm esteve (~quassel@cli-5b7ee22f.bcn.adamo.es) |
| 15:26.53 | *** join/#storm vvinet (~vince@2001:470:8829:1000:227:eff:fe25:ee64) |
| 15:34.57 | *** join/#storm vvinet_ (~vince@2001:470:8829:1000:227:eff:fe25:ee64) |
| 15:36.13 | *** join/#storm vvinet_ (~vince@2001:470:8829:1000:227:eff:fe25:ee64) |
| 15:48.28 | *** join/#storm magmatt (~Matt@74-92-245-181-Utah.hfc.comcastbusiness.net) |
| 15:59.20 | *** part/#storm magmatt (~Matt@74-92-245-181-Utah.hfc.comcastbusiness.net) |
| 16:29.27 | *** join/#storm webmaven (~webmaven@97.65.119.2) |
| 16:37.12 | *** join/#storm fcorrea (~fcorrea@187.3.22.136) |
| 16:58.02 | *** join/#storm jdobrien (~john@adsl-235-204-136.mco.bellsouth.net) |
| 17:17.31 | *** join/#storm fcorrea (~fcorrea@187.3.22.136) |
| 17:45.02 | *** join/#storm keppla (~keppla@i577B09DB.versanet.de) |
| 17:49.49 | *** join/#storm _mup_ (~mup@li37-90.members.linode.com) |
| 18:26.38 | *** join/#storm esteve (~quassel@cli-5b7ee22f.bcn.adamo.es) |
| 18:47.08 | *** join/#storm cody-somerville (~cody-some@ubuntu/member/somerville32) |
| 18:47.26 | cody-somerville | Hey. I was wondering if a storm developer has had a chance to review LP #738035 yet. |
| 18:47.28 | _mup_ | Bug #738035: ProgrammingError raised instead of DisconnectionError <oem-services> <Storm:New> < https://launchpad.net/bugs/738035 > |
| 18:47.49 | *** join/#storm fcorrea (~fcorrea@187.3.22.136) |
| 18:52.44 | therve | cody-somerville, it looks like the message you got is not in the list of messages handled? |
| 18:53.44 | cody-somerville | oh! I misread the code. Yea, that would explain it. |
| 18:55.24 | therve | it looks like I could just commit that as trivial |
| 18:57.41 | cody-somerville | Do you know if there is a bug upstream in psycopg2 to make detecting/handling disconnections more reliable/robust? |
| 18:58.06 | therve | no idea |
| 18:58.27 | therve | I think things improved already, I don't know if there is an ongoing effort |
| 19:00.32 | _mup_ | storm/trunk r388 committed by thomas@canonical.com |
| 19:00.32 | _mup_ | Add another message to be detected as disconnection error in Postgres, reported |
| 19:00.32 | _mup_ | by Cody [f=738035] [trivial] |
| 19:21.23 | *** join/#storm lifeless (~robertc@122-63-10-108.jetstream.xtra.co.nz) |
| 20:18.21 | *** join/#storm cody-somerville (~cody-some@d67-193-72-198.home3.cgocable.net) |
| 20:18.21 | *** join/#storm cody-somerville (~cody-some@ubuntu/member/somerville32) |