Conversation

Notices

1. Token (coin@asimon.org)'s status on Tuesday, 13-Feb-2024 04:22:22 JST Token

   in reply to

   • Alex Gleason
   • feld
   @alex @feld Did you change the main development branch from 'develop' to 'main' now. Is that what we're using now?
   • feld (feld@bikeshed.party)'s status on Tuesday, 13-Feb-2024 04:22:23 JST feld

     in reply to

     • Alex Gleason
     @alex oh very nice, however we were going to add a job to automatically delete old notifications as a fix too haha
     
     I guess we can still do that but limit the retained notifs to 1000 instead of unbounded
   • Alex Gleason (alex@gleasonator.com)'s status on Tuesday, 13-Feb-2024 04:22:23 JST Alex Gleason

     in reply to

     • feld
     @feld You'll still want to patch, because the code doesn't match the index. I ran "explain analyze" on it like a big boy and figured it out. 😃
   • Alex Gleason (alex@gleasonator.com)'s status on Tuesday, 13-Feb-2024 04:22:24 JST Alex Gleason

     in reply to

     • Alex Gleason
     • feld
     @feld check ☝️
   • Alex Gleason (alex@gleasonator.com)'s status on Tuesday, 13-Feb-2024 04:22:25 JST Alex Gleason
     I have a rare performance fix for Pleroma that will fix some issues resulting from having a very large notifications table: https://gitlab.com/soapbox-pub/rebased/-/merge_requests/301
     
     tl;dr an index already existed but the code wasn't using it