Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 11:05
    CaselIT milestoned #9197
  • 11:05
    CaselIT labeled #9197
  • 11:05
    CaselIT labeled #9197
  • 11:05
    CaselIT opened #9197
  • 11:05
    CaselIT labeled #9197
  • 00:25
    sqlalchemy-bot closed #9182
  • 00:25

    sqlalchemy-bot on main

    apply of_type error message to … (compare)

  • Jan 30 22:57
    sqlalchemy-bot closed #9173
  • Jan 30 22:57

    sqlalchemy-bot on main

    don't count / gather INSERT bin… Merge "don't count / gather INS… (compare)

  • Jan 30 22:56
    FrozzWay edited #9194
  • Jan 30 22:56
    FrozzWay edited #9194
  • Jan 30 22:45
    zzzeek labeled #9194
  • Jan 30 22:45
    zzzeek closed #9194
  • Jan 30 22:00
    CaselIT labeled #9194
  • Jan 30 22:00
    CaselIT unlabeled #9194
  • Jan 30 21:19
    FrozzWay edited #9194
  • Jan 30 21:18
    FrozzWay labeled #9194
  • Jan 30 21:18
    FrozzWay opened #9194
  • Jan 30 19:27
    CaselIT edited #9193
  • Jan 30 17:51
    iOlenka opened #19
mike bayer
@zzzeek
ok my weakref idea does not work, correct
i dont like the ida of using weakrefs, someone using this (which they shouldnt) should make sure their task does .remove()
Federico Caselli
@CaselIT
I agree. so maybe a dragon?
Jonathan Vanasco
@jvanasco
there be dragons.
Federico Caselli
@CaselIT
there be async dragons in this case
Federico Caselli
@CaselIT

" send in the dragons on async_scoped_session " sqlalchemy/sqlalchemy@ac46321

ahahah

Alex Grönholm
@agronholm
yup, scoped sessions are problematic exactly because people don't think to close them...just the other day I was advising somebody who thought scoped sessions were "automatically managed" and didn't need to be closed.
2 replies
Federico Caselli
@CaselIT
@zzzeek the dataclasses stuff seems pretty easy to extend
mike bayer
@zzzeek
@CaselIT im sure it's not too hard as long as you arent doing slots
Federico Caselli
@CaselIT
yes, I think slots would require support in the orm
since you would need to add the _sa_... slot too
Federico Caselli
@CaselIT
but since there is basically no change compared with a plain class, I think we can live without supporting them
mike bayer
@zzzeek
I got ranges and multiranges working on asyncpg etc
was MUCH easier than I thought, psycopg has lots of complex stuff that isn't needed
@CaselIT re: slots, yes we'd need a big attribute change, since the InstanceState needs to have a "dict", so it would need to know how to get a "dict" from a slots class
@CaselIT all very doable, but a lot of work and also InstanceState kind of defeats the "performance" goal
@CaselIT having a "dict" over a slots class has to be a cython thing otherwise it's much slower than not using slots
Federico Caselli
@CaselIT
I'm really not sure it's worth it, since the performance change is negligible compared with a normal class
mike bayer
@zzzeek
just tried to release and twine upload seems to be silently failing
Jonathan Vanasco
@jvanasco
is it maybe from that new "protected projects" thing?
mike bayer
@zzzeek
@jvanasco seems very likely
Federico Caselli
@CaselIT
Strange, we use a token to upload
the github action haa still not run. lets see if that fails, but in any case its strange that twine is happy but nothing happens
Federico Caselli
@CaselIT
I think there is something strange going on on pypi side. This errors https://test.pypi.org/project/SQLAlchemy/
Federico Caselli
@CaselIT
It seem to be working now
mike bayer
@zzzeek
they pushed a front-end thing that broke the view
so we were seeing cdn cached
and twine upload happily says nothing if file already exists
Federico Caselli
@CaselIT
Well at least it's not our fault
mike bayer
@zzzeek
nope
it's jsut twine's silent failure made me unsure if upload was just afiling
instead of dumb UX bug
Federico Caselli
@CaselIT
Still twine could say something when the file already exists even if it still exists with success
mike bayer
@zzzeek
yes
Federico Caselli
@CaselIT
we seem to have comments also on mssql
mike bayer
@zzzeek
hey folks just making berakfast will be there in 5
Gord Thompson
@gordthompson

we seem to have comments also on mssql

@CaselIT - Where?

Federico Caselli
@CaselIT
hi all
Gord Thompson
@gordthompson
hey
Federico Caselli
@CaselIT
Gord Thompson
@gordthompson
oh, ok. I thought you meant comments in an issue/discussion.
Federico Caselli
@CaselIT
nope
Gord Thompson
@gordthompson
The thing about table comments, etc. in mssql is that nobody uses them.
Federico Caselli
@CaselIT
btw, maybe this could be an alternative to zimports that let us avoid maintaining it? https://github.com/asottile/reorder_python_imports

The thing about table comments, etc. in mssql is that nobody uses them.

you have to do wired things to use them

Gord Thompson
@gordthompson
Perhaps I should have said "nobody uses them with DDL", esp. because of the "weirdness". If you add a comment in SSMS then you're already using SSMS so it's way easier to just change it there. (Although I must confess that I don't use them myself so I don't know how good SSMS is at managing such things.)
mike bayer
@zzzeek
awrite
Federico Caselli
@CaselIT
hey mike
mike bayer
@zzzeek
@gordthompson nobody uses 90% of the silly stuff we support