Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 13:18
    zzzeek milestoned #8579
  • 13:18
    zzzeek unlabeled #8579
  • 13:18
    zzzeek labeled #8579
  • 13:03
    zzzeek milestoned #8578
  • 13:03
    zzzeek unlabeled #8578
  • 13:03
    zzzeek labeled #8578
  • 13:02
    zzzeek labeled #8578
  • 13:02
    zzzeek labeled #8578
  • 12:57
    sqlalchemy-bot closed #8577
  • 12:57

    sqlalchemy-bot on main

    add typing for sqlalchemy.orm.v… Merge "add typing for sqlalchem… (compare)

  • 12:57

    sqlalchemy-bot on main

    adjust tests for sqlites w/o re… (compare)

  • 11:17
    Rolfdv opened #8580
  • 11:15
    Rolfdv labeled #8579
  • 11:15
    Rolfdv labeled #8579
  • 11:15
    Rolfdv opened #8579
  • 03:43
    acnebs edited #8578
  • 03:41
    acnebs edited #8578
  • 03:39
    acnebs edited #8578
  • 03:38
    acnebs edited #8578
  • 03:37
    acnebs labeled #8578
mike bayer
@zzzeek
@gordthompson yes there's that but also the crapton of new features added too
Federico Caselli
@CaselIT
I don't remember did we remove everything deprecated already for v2?
like I think we could remove everything that's deprecated since before 1.3?
mike bayer
@zzzeek
I think we did a pass of that nature ? maybe not pinned at 1.3?
Federico Caselli
@CaselIT
for 1.4
we removed up to v1
in any case next thing I wanted to look into was the mysql PR that's stalled since a while sqlalchemy/sqlalchemy#5536
and the scalar value for postgresql
mike bayer
@zzzeek
yeah
whats the scalar value for postgresql ?
Federico Caselli
@CaselIT
mike bayer
@zzzeek
OK , we shoudl be careful on that one to not break things
im confident we have enoug test coverage
Federico Caselli
@CaselIT
I think the idea was to do a values().scalar_valued() o something like this
mike bayer
@zzzeek
yes
Federico Caselli
@CaselIT
so it should not break stuff, since it's a new thing
mike bayer
@zzzeek
OK , also that the new API makes sense, etc. havent looked in awhile
Federico Caselli
@CaselIT
basically it's so that I can remove a custom thing I have at work :)
mike bayer
@zzzeek
yes
Federico Caselli
@CaselIT
I guess another thing to look into is the cibuildwheel
since betas are nearer
also github has added support for arm osx
mike bayer
@zzzeek
I guess, I think we just have to pull the switch on that b.c. it will take effect for 1.4 at the same time
so lets just do it
Federico Caselli
@CaselIT
I think we can just do v2
let's leave 1.4 as is
since IIRC it does not support py2
mike bayer
@zzzeek
right but the release actions always come from the main branch right?
or are they broken up ?
Federico Caselli
@CaselIT
no, rel1_4 runs the 1.4 one
mike bayer
@zzzeek
oh great, we should just update now in main
Federico Caselli
@CaselIT
yes, I've it on my list
Federico Caselli
@CaselIT
@gordthompson for mssql do you have suggestion on how to make it support unicode?
Federico Caselli
@CaselIT
nevermind, I just needed to tell the compiler to translate it as nvarchar
Jonathan Vanasco
@jvanasco
@CaselIT is this for a PR or usage scenario?
Federico Caselli
@CaselIT
it was for a PR on mssql comments. but I've solved it
Jonathan Vanasco
@jvanasco
thanks. i'll look out for that on gerrit. i was working on a PR a few months ago and ran into a similar issue.
Federico Caselli
@CaselIT
I've milestoned a few issues that were without a milestone
mike bayer
@zzzeek
@CaselIT yeah I'm amazed these were hanging around
Federico Caselli
@CaselIT
I guess we forgot them
Federico Caselli
@CaselIT
mike for asyncio maybe we could add options to refresh so that if an object was loaded without relationship it could be refreshed taking into account the new options?
or should we point users to get with populate_existing=true in that case?
mike bayer
@zzzeek
i would think this has come up before is there any issue talkinga bout it?
Federico Caselli
@CaselIT
no, a question in the other gerrit
mike bayer
@zzzeek
"For more open ended "refresh" functionality,
including the ability to refresh the attributes on many objects at
once while having explicit control over relationship loader
strategies, use the
:ref:populate existing <orm_queryguide_populate_existing> feature
instead."
i think that's the current direction
Federico Caselli
@CaselIT
than I guess it has already come up
mike bayer
@zzzeek
like refresh() doesnt really do anything querying can't do. it precedes populate_existing
Federico Caselli
@CaselIT
I think it makes sense keeping it like this, so we avoid having two ways of doing the same thing
mike bayer
@zzzeek
yes if we keep adding options to refresh() then we are duplicating effort