| 00:17.07 | *** join/#storm artista_frustrad (n=artista_@201-24-231-164.ctame704.dsl.brasiltelecom.net.br) |
| 02:00.45 | *** join/#storm drudi (n=drudi@189.121.88.99) |
| 02:22.45 | *** join/#storm jdobrien (n=john@132.199.119.70.cfl.res.rr.com) |
| 04:17.06 | *** join/#storm artista_frustrad (n=artista_@201-24-230-174.ctame704.dsl.brasiltelecom.net.br) |
| 04:24.04 | *** join/#storm stub (n=stub@canonical/launchpad/stub) |
| 04:25.52 | jkakar | jamesh, stub: Is there anything not in this list that you guys thing we need before we can cut 0.15? https://edge.launchpad.net/storm/+milestone/0.15 |
| 04:26.11 | stub | I've already targetted everything I need to 0.15 |
| 04:26.14 | jkakar | Cool. |
| 04:26.16 | jkakar | Me too. |
| 04:26.55 | stub | And the resultset aggregate is the only thing I *need* |
| 04:27.04 | jamesh | jkakar: there is one: I need to remove the check in storm.django to make sure the transaction middleware is configured |
| 04:27.28 | jamesh | we have a django app where we don't want request-duration transactions |
| 04:27.37 | jkakar | Ah, okay. |
| 04:27.45 | jkakar | jamesh: Is there a bug filed for it yet? |
| 04:28.01 | jamesh | jkakar: yeah, but it is currently set private. |
| 04:28.06 | jkakar | jamesh: Okay. |
| 04:28.46 | jamesh | the change is basically just to remove the first conditional from storm.django.stores.configure_stores() |
| 04:28.54 | jkakar | jamesh: This fix for the result set aggregates is nice, simpler than I expected. Thanks. |
| 04:31.41 | jamesh | stub: If you want, I'll add you to the Storm developers' team so you can merge your branch. |
| 04:31.58 | stub | Ok. I can add myself if you like ;) |
| 04:32.16 | jamesh | actually, I can't add you -- I'm not an administrator |
| 04:32.30 | jamesh | I'm sure niemeyer won't mind if you add yourself |
| 04:32.50 | stub | I'd rather not step on any toes ;) |
| 04:33.02 | jkakar | tries |
| 04:33.14 | jamesh | looks like only niemeyer and therve are admins |
| 04:33.34 | jkakar | stub: Just add yourself. :) |
| 04:33.49 | stub | So is jamesh's fix landing then? I'll collect the Launchpad test failures then for a closer look |
| 04:35.11 | jamesh | stub: the distinct aggregates branch will need a second review after jkakar's |
| 04:35.11 | jkakar | stub: I've just approved lp:~jamesh/storm/bug-217644. It needs another review before it can land (hint hint). :) |
| 04:35.50 | stub | It confuses me - it gets into the bits of Storm I have no clue about. I would have reviewed it already otherwise. |
| 04:35.56 | jkakar | stub: Okay. |
| 04:36.05 | jamesh | stub: the test failures might be due to it fixing more than just distinct aggregates. |
| 04:36.16 | stub | It looked competent though if that is good enough ;) |
| 04:36.45 | jamesh | it'll also change the result of aggregates over result sets with limit or offset clauses |
| 04:36.47 | stub | I expect it will be Launchpad's fault rather than more edge cases |
| 04:37.57 | jamesh | e.g. store.find(Foo).order_by(Foo.id)[:10].max(Foo.id) would now give you the maximum Foo.id in the first 10 rows rather than in all rows |
| 04:38.01 | stub | I'll still kick off the test suite now - might as well see what differences we get between the 0.14 and trunk+fix |
| 04:39.06 | jkakar | I'm running the Landscape test suite with the aggregate branch, to see if we have any breakage as a result. |
| 04:46.34 | jkakar | All tests pass here with the trunk+fix. |
| 04:50.23 | jkakar | stub: Is the ZStorm factory branch reviewable or are you still trying to decide what you want to do with it? |
| 04:50.59 | stub | It needs to be reworked. If it misses this release I'm not too fussed. |
| 04:51.12 | stub | Unless you want to stand up for my design, which I personally prefer ;) |
| 04:51.14 | jkakar | stub: Okay. |
| 04:52.03 | jkakar | stub: I'll check out the branch in its current state, along with jamesh's feedback, and see what I make of it. |
| 04:57.09 | mup | storm/trunk r315 committed by jkakar@kakar.ca |
| 04:57.09 | mup | Merged coalesce-expr [r=jamesh,niemeyer] [f=403796] |
| 04:57.09 | mup | A new Coalesce expression is available in storm.expr. |
| 05:42.00 | *** join/#storm jukart (i=lovely@81.189.156.94) |
| 05:47.23 | *** join/#storm elmom (n=elmom@hoasnet-ff05dd00-13.dhcp.inet.fi) |
| 06:09.23 | *** join/#storm wgrant (n=wgrant@ubuntu/member/wgrant) |
| 07:01.27 | *** join/#storm goschtl (n=goschtl@p5B0BFA95.dip.t-dialin.net) |
| 08:02.44 | *** join/#storm fcorrea (n=fcorrea@189.62.16.54) [NETSPLIT VICTIM] |
| 08:18.08 | *** join/#storm artista_frustrad (n=artista_@201-15-205-162.ctame704.dsl.brasiltelecom.net.br) |
| 10:22.32 | *** join/#storm zmijunkie (n=chatzill@92.117.26.112) |
| 11:03.15 | *** join/#storm drudi (n=drudi@189.121.88.99) |
| 11:26.35 | *** join/#storm andrea-bs (n=andrea@ubuntu/member/beeseek.developer.andrea-bs) |
| 12:18.04 | *** join/#storm artista_frustrad (n=artista_@201-15-251-158.ctame704.dsl.brasiltelecom.net.br) |
| 12:25.45 | *** join/#storm jdobrien (n=john@132.199.119.70.cfl.res.rr.com) |
| 12:32.55 | *** join/#storm drudi (n=drudi@189.64.199.115) |
| 12:34.50 | *** join/#storm niemeyer (n=niemeyer@200-138-49-140.ctame705.dsl.brasiltelecom.net.br) |
| 13:25.01 | *** join/#storm oubiwann (n=oubiwann@209.104.244.164) |
| 13:39.11 | *** join/#storm oubiwann (n=oubiwann@209.104.244.164) |
| 14:40.55 | *** join/#storm sidnei_ (n=sidnei@201-40-159-53.cslce701.dsl.brasiltelecom.net.br) |
| 14:53.53 | *** join/#storm drudi (n=drudi@189.116.142.114) |
| 15:50.57 | *** part/#storm goschtl (n=goschtl@p5B0BFA95.dip.t-dialin.net) |
| 15:59.14 | *** join/#storm andrea-bs (n=andrea@ubuntu/member/beeseek.developer.andrea-bs) |
| 16:18.38 | *** join/#storm artista_frustrad (n=artista_@201-15-219-236.ctame704.dsl.brasiltelecom.net.br) |
| 16:41.58 | *** join/#storm jdobrien (n=john@132.199.119.70.cfl.res.rr.com) |
| 16:45.02 | *** join/#storm tomiko (n=jfcogato@89.141.110.101.dyn.user.ono.com) |
| 16:45.19 | tomiko | hello, may I shot a question? :) |
| 16:45.58 | tomiko | I'm tryng to make a debian package with debian |
| 16:46.27 | tomiko | anybody know the depends of storm ? :) |
| 16:46.37 | tomiko | mysql, sqlite ... ? |
| 16:46.59 | tomiko | python for sure, but once more? ^o^ |
| 16:48.49 | jkakar | tomiko: The database-specific dependencies are only required if you want to use Storm with a particular database. |
| 16:49.20 | jkakar | tomiko: For MySQL you'll want to install python-mysqldb and for PostgreSQL you'll want python-psycopg2. |
| 16:49.48 | tomiko | mmmm ok, only the db dependences |
| 16:49.58 | tomiko | thanks, thats what I supposed ;) |
| 16:50.06 | mup | storm/ppa-build r306 committed by jkakar@kakar.ca |
| 16:50.06 | mup | - Merged trunk. |
| 16:51.02 | jkakar | tomiko: If you're interested, I have a branch that makes Storm buildable in a PPA. I haven't tested it recently, but the branch is at lp:~jkakar/storm/ppa-build. |
| 16:51.50 | tomiko | PPA? |
| 16:53.11 | jkakar | tomiko: Personal Package Archive |
| 16:53.17 | jkakar | tomiko: It's a build service provided by Launchpad. |
| 16:53.48 | jkakar | tomiko: There's more information about them here: https://help.launchpad.net/Packaging/PPA |
| 16:54.24 | tomiko | and with that PPA can I make an easy debian package? :) |
| 16:54.40 | *** join/#storm lfaraone (n=LukeFara@ubuntu/member/lfaraone) |
| 16:54.50 | lfaraone | Hey, does Storm run in Etch? |
| 16:54.52 | tomiko | I really hate that stuffs jeje, but I have to do it, to make a package of a big project that use storm :) |
| 16:55.32 | sidnei_ | lfaraone: it runs on windows, so it must run on etch *wink* |
| 16:56.24 | lfaraone | sidnei_: heh, FTBFS when I try to use the egg. |
| 16:56.50 | lfaraone | Scratch that, seems to work. |
| 16:57.02 | jkakar | tomiko: I don't think so, sadly... PPAs currently only build Ubuntu packages. |
| 16:58.30 | tomiko | yeath that what I'm reading right now in the web you gave me ... |
| 16:59.07 | tomiko | but now I have the dependencies, so thanks for your help jkakar , when I have the .deb I publish in some place ;) |
| 16:59.22 | tomiko | byeeeee byeeeeee dude! |
| 16:59.29 | jkakar | tomiko: np. :) |
| 17:06.47 | lfaraone | Is there a way to store an object that doesn't make use of storm objects in storm? |
| 17:09.31 | jkakar | lfaraone: I don't think I understand the question... |
| 17:09.57 | jkakar | lfaraone: Do you want to put a non-Storm object in a database with Storm, or do you want to store a non-Storm object as an attribute of a Storm object? |
| 17:10.29 | jkakar | lfaraone: If the former, then there's no real support for that, if the latter, you could use the Pickle field to store your non-Storm object as a blob. |
| 17:11.37 | lfaraone | jkakar: The former, and I had a feeling that was the answer. |
| 17:19.36 | jkakar | lfaraone: You can always use store.execute to run whatever query you want, to push data in and pull it out, but you won't get the benefits of Storm's caching or other features by going that route. |
| 17:20.05 | *** join/#storm andrea-bs (n=andrea@ubuntu/member/beeseek.developer.andrea-bs) |
| 17:28.56 | *** join/#storm jukart (n=jukart@d86-32-163-51.cust.tele2.at) |
| 17:37.58 | *** join/#storm zmijunkie (n=chatzill@la-pool-91-204-5-249.free.de) |
| 18:02.12 | *** join/#storm jukart (n=jukart@d86-32-163-51.cust.tele2.at) |
| 19:01.44 | lfaraone | jkakar: Not sure if this is supposed to be obvious, but how do I connect to an already extant store and database? (I get how to use create_database and store(), but is there a connect_database()?) |
| 19:02.58 | jkakar | lfaraone: You don't need to do anything explicit. |
| 19:03.12 | jkakar | lfaraone: Just do Store(create_database("uri")) and then start using the store. |
| 19:03.19 | jkakar | lfaraone: Storm will automatically connect for you. |
| 19:03.41 | lfaraone | jkakar: Will that retrieve data already in the store from prior sessions? |
| 19:03.55 | jkakar | lfaraone: If, for whatever reason, the connection is dropped, you'll get a DisconnectionError exception which you can catch and deal with. |
| 19:04.12 | jkakar | lfaraone: Not implicitly, you'll have to run a query to get data in the database. |
| 19:04.38 | jkakar | lfaraone: The basic workflow is: (1) create a Store, (2) create a (Storm) object and add it to the store with store.add and (3) commit changes to the database with store.commit. |
| 19:04.58 | jkakar | lfaraone: You can then use store.find to get the object(s) that already exist in the database. |
| 19:06.40 | lfaraone | jkakar: Makes sense. Sorry to bother you with all the questions/ |
| 19:06.56 | jkakar | lfaraone: No bother at all. Keep 'em coming. :) |
| 19:53.52 | lfaraone | jkakar: When I attempt to assign to a string I had previously defined as Unicode(), I get this: |
| 19:53.59 | lfaraone | <PROTECTED> |
| 19:54.12 | lfaraone | (replacing `'my.jobsource'` with whatever) |
| 19:54.24 | lfaraone | jkakar: I'm on Python 2.4. Is there something I'm doing wrong? |
| 20:11.07 | lfaraone | (nevermind, just switched to unicode()ing all strings before I assigned them) |
| 20:26.56 | *** join/#storm artista_frustrad (n=artista_@201-35-6-239.ctame704.dsl.brasiltelecom.net.br) |
| 20:38.56 | jkakar | lfaraone: Storm doesn't do any implicit conversions, so if you use a Unicode column you need to pass a unicode value to it. |
| 20:39.26 | jkakar | lfaraone: It can be a bit of a pain at a time, but this behaviour is actually a very good thing because it means Storm won't silently corrupt your data. :) |
| 20:39.27 | *** join/#storm salgado (n=salgado@canonical/launchpad/salgado) |
| 21:41.25 | *** join/#storm drudi (n=drudi@187.46.155.66) |
| 21:52.45 | jkakar | salgado: Heya. Did you mean to change bug #374909 from 'Fix committed' to 'Triaged'? It looks like the fix did indeed land in r308...? |
| 21:52.46 | mup | Bug #374909: InterfaceError: connection already closed should be converted into DisconnectionError <Storm:Triaged by jamesh> <https://launchpad.net/bugs/374909> |
| 21:53.15 | salgado | jkakar, I just commented there why I think it's not fixed |
| 21:53.52 | salgado | we were still seeing some InterfaceErrors in launchpad.net and after some digging around I noticed Connection.rollback() is not treating them as disconnection errors |
| 21:54.13 | jkakar | salgado: Oh, I just got it... missed it originally because of email lag. |
| 21:54.20 | salgado | may be a different bug, though? |
| 21:54.37 | jkakar | salgado: I think we should keep that bug closed, since the fix is already merged and open a new one, but it's not a big deal. I'll check out your proposed fix sometime today. |
| 21:55.10 | salgado | jkakar, cool, just let me know and I can file a new one if necessary |
| 22:51.06 | *** join/#storm oubiwann_ (n=oubiwann@209.104.244.164) |