<yellowhatpro[m]> "btw I wanted to discuss a better..." <- > <@yellowhatpro:matrix.org> btw I wanted to discuss a better polling logic than the one I am currently using.... (full message at <https://matrix.chatbrainz.org/_matrix/media/v3/...>)
zas[m]: looks like it corresponds to a lot of data received on gaga too
I didn't find anything abnormal in MB website logs
yvanzo[m]
bitmap: The peak time matches the peak time of yesterday.
I thought that it was due to the MBS release but maybe it was unrelated.
yellowhatpro[m]
<bitmap[m]> "> <@yellowhatpro:matrix.org> btw..." <- it's just a corner case I am thinking of. Suppose we started polling with id 100 for edit_note, so the next poll currently begins with 110. But what if in the next poll, the edit_note id is 105, that means, in the current poll, we were only able to process 5 rows, and no new rows were added in edit_note.
yvanzo[m]
yellowhatpro: Retain the last polled edit_note ID and begin the next poll with ID + 1?
bitmap[m]
yes, that's also what I was suggesting
I didn't understand why you need a function to count the rows
yvanzo[m]
exactly, just in other words, maybe I don’t understand either the question
yellowhatpro[m]
umm, but in this case, what if no new row is being added to edit_note, and we are still doing a redundant +1 on each poll?
bitmap[m]
if it doesn't poll anything then you're not incrementing the counter
yellowhatpro[m]
Alright got it.
One more thing to ask though, where do we store if a user is marked spammer?
yvanzo[m]
yellowhatpro: It is marked in the column `privs` of the table `editor`.
It is also matching the peak in MB website response time yesterday.
However those peaks are not the most busy periods for gaga in the two last days.
bitmap[m]
yeah, it could be unrelated, or it depends on the type of query it's performing (not sure what tasks run there)
akshaaatt[m] has quit
BrainzGit
[listenbrainz-server] 14anshg1214 merged pull request #2920 (03master…dashboard-ws-listens): LB-1597: 'New listens since you arrived' carries over between profiles https://github.com/metabrainz/listenbrainz-serv...
[musicbrainz-server] 14mwiencek opened pull request #3298 (03master…update-db-privileges-improvements): More UpdateDatabasePrivileges.pl improvements https://github.com/metabrainz/musicbrainz-serve...
<kellnerd[m]> "https://tabler.io/icons/icon/..." <- that kinda feels like we've... "arrived". only took nearly 25 years, lol
monkey: that has got to feel pretty good for you, no?
monkey[m]
Absolutely
monkey[m] uploaded an image: (14KiB) < https://matrix.chatbrainz.org/_matrix/media/v3/download/chatbrainz.org/lPGtJPubWlxKdKcuppfQHUPN/image.png >
aerozol[m]
Thanks for replying mayhem! I'm sure you've seen the follow up. Have fun 😛
FWIW I think I briefly raised that "required" might not be the right wording in this context. But then again, I understand that we need the support or RIP. Not an easy discussion
mayhem[m]
aerozol: just responded.
I think open source purists have a hard time with our model. more reasonable people don't.
reosarevok[m]
What's the discussion, something on mastodon/reddit/whatsit?
In the sense that someone who doesn't actually understand the legal stuff might think it's a legal rather than moral requirement
But eh, if it works
"strongly encouraged" I'm sure would make people be like "oh, so it's free, neat" who shouldn't so
mayhem[m]
"works in practice, is a nightmare in theory"
reosarevok[m]: The wording is very carefully chosen. And I could choose clearer words in such tweets, but that cuts though the little bit of confusion we allow ourselves. A bit of a grey zone is a good thing.
in this case.
yvanzo[m]
BrainzBot: UPDATE:JIRA
BrainzBot
Successfully updated projects list
reosarevok[m]
yvanzo: added watermark and raw/unedited now for events too