#metabrainz

reosarevok: testing with prod data and this medium_track_durations join is turning out to be really slow, but I have a way around it: https://github.com/metabrainz/musicbrainz-serve...

https://test.musicbrainz.org/ is updated with schema changes

https://test.musicbrainz.org/artist/38712b4c-0f... now loads faster on the slow test server than it does from prod (so it should be *really* fast from prod :))

bitmap: thanks!

ruaok, something seems to have broken https://stats.metabrainz.org/d/OGg5QUCGz/listen...

Yeah, i think I know what it is.

Was there something that needed merging that I missed?

no i don't think so. i'll check the metric writer logs to see what went wrong.

Does anyone know of an MB release offhand without a medium?

Project bookbrainz-site build #3968: errored in 1 min 43 sec: https://travis-ci.org/bookbrainz/bookbrainz-sit...

zas, yvanzo: https://twitter.com/Nomys_Tempar/status/1393474...

Freso: IIRC we have a report for that, but I assume you've found one by now

ruaok, ^, looks like the error did not get logged. probably that's why sentry didn't see it so no report.

[musicbrainz-server] reosarevok opened pull request #2103 (schema-change-2021-q2…privaciesDiffer): Fix partially renamed variable https://github.com/metabrainz/musicbrainz-serve...

bitmap: ^ whoops

Not SQL related at least :)

Feel free to merge

good catch

[musicbrainz-server] mwiencek merged pull request #2103 (schema-change-2021-q2…privaciesDiffer): Fix partially renamed variable https://github.com/metabrainz/musicbrainz-serve...

bitmap: hmm. I changed one release in https://test.musicbrainz.org/release-group/4861... from promo to official

It still doesn't seem to be showing on https://test.musicbrainz.org/artist/b7ffd2af-41...

lemme see

Is there some cache that is not being invalidated maybe?

hmm. the ID for the release group is still in artist_release_group_pending_update

maybe a trigger is not installed

ok somehow these triggers got lost in a rebase or something...I'll restore them

weird

Guess it's good I tested, lol

yeah

and the pgtap tests got unstaged somehow too which is why circleci passed. ugh

reosarevok: installed those triggers to test.mb, so it should update if you change teh status again

pushed fixes for that to https://github.com/metabrainz/musicbrainz-serve...

Wonder what else we should be testing

artist series?

or did we still need code changes for that

I guess other than that, place ratings, collection merging, maybe updating some cardinalities/frequencies

Collection merging works

I'll test the rest tomorrow

ok, I just sent the upgrade google doc I've been workin on too

Freso: ruaok: oh I'm not sure this was discussed last week. are we having a meeting on monday? iirc we cancelled it last schema change day

since we started the upgrade at standard meeting time. (that's tbd, I don't mind starting earlier either)

By all means, let's cancel it. Bake it so.

Or is it.. "it is so baken"?

it is forthwith so bakened

[musicbrainz-android] akshaaatt opened pull request #71 (master…tagger_update): Tagger update https://github.com/metabrainz/musicbrainz-andro...