#musicbrainz-devel

/

      • ocharles
        I'm going to go down the no email means it's all ok route
      • djce
        k
      • reosarevok joined the channel
      • oh, ocharles can I borrow you for 3 min on a different topic? ping me when ready....
      • ocharles
        djce: fire away
      • djce
        ok. Just wanted to briefly discuss http://tickets.musicbrainz.org/browse/MBH-153
      • I updated the spec on that ticket
      • voiceinsideyou joined the channel
      • and I think it'll mainly (probably only) affect the MBS codebase in fact.
      • ocharles
        yay, md5sums
      • djce
        So some combination of a FYI, and a "please shout if you see a problem",
      • and my next task is to estimate how long that'll take
      • ocharles
        Yea, it looks like it's just MBS
      • djce
        so I can keep my boss updated as to how long I'm bunking off BBC work to do MBH instead :-)
      • ocharles
        :)
      • djce
        I'm still unclear as to where I'll be testing this, alas.
      • Might have to resurrect replication packet emitting on hobbes.
      • but I have no access to any MB slaves :-(
      • ocharles
        hobbes is pretty much ready for that I'd think
      • djce
        (well.... hmm... I could use pretty much any machine in fact if I just mock the bit where it loads the packet into the database I suppose).
      • ok. Thanks. that's all for now, just wanted to keep you informed mainly.
      • (damn, 10 sec over my 3 minute chat estimate ;-)
      • ocharles
        lol
      • jdamcd joined the channel
      • voiceinsideyou joined the channel
      • cjk32 joined the channel
      • cjk32
        ISE on the release editor, MBS-3165 if someone wants to capture stack traces etc from the error log.
      • ocharles
        Sigh, we're getting 502s again it seems
      • jdamcd joined the channel
      • voiceinsideyou joined the channel
      • warp
        goodmorning!
      • cjk32: regarding the session timeout, it's probably better to submit what you have, and add an editnote saying you will add the remaining discs later.
      • still, it shoudln't "internal server error" ofcourse
      • voiceinsideyou joined the channel
      • voiceinsideyou joined the channel
      • ocharles
        PostgreSQL indexing and plperl just blew me away
      • I can find all edit relationship edits that use a certain relationship type in....
      • 0.36ms
      • luks
        planning to use plperl in MB?
      • ocharles
        only for this json searching
      • i can't see any other way to query it fast otherwise
      • bitmap_ joined the channel
      • adhawkins
        Does the android app use /ws/2 or /ws/1?
      • If I search for release 'Aerial' and select the Kate Bush one, it only lists 5 matches, all for 'A Sky of Honey'. None for the other disc in the release
      • The matches all have 16 tracks, whereas in reality it's two discs, one with 7, the other with 9.
      • ocharles
        warp: http://codereview.musicbrainz.org/r/1450/diff/#... could you explain the last hunk?
      • echo "SELECT plainto_tsquery('mb_simple', '☺')" | ./admin/psql READWRITE just gives "NOTICE: text-search query contains only stop words or doesn't contain lexemes, ignored"
      • warp
        ocharles: when in the release editor you add a track to a VA release, the artist credit for that track will be empty.
      • ocharles: if you submit that you will get a warning that the track needs an artist
      • ocharles: but without these changes you can never submit a new artist, because while iterating over each credit 'current' will be falsy (because it is empty) for each credit.
      • ocharles
        hrm
      • luks
        ocharles: not very easy to fix, we always had that problem (mainly with !!! :))
      • ocharles
        well, I don't really get it but if it works the rest looks ok :)
      • luks: yea
      • I wondered if there was a way to just do a direct text comparison as well
      • warp
        ocharles: hrm, perhaps it would be cleaner and clearer to just compare to.names.length with from.names.length
      • luks
        well, you can do "... OR artist_name.name = ?" :)
      • warp
        ocharles: before the $.each() over to.names.
      • ocharles
        luks: yea, that's probably what I will be doing
      • luks
        but it definitely doesn't look like it needs to get fixed in 2011-08-08
      • ocharles
        but that should presumably rank at the very top too
      • well, it's in there :)
      • luks
        warp put it there
      • ocharles
        I asked for other work yesterday, and didn't really get any
      • yea
      • luks
        it's a very minor issue
      • warp wouldn't consider it minor.
      • MB had it since the beginning
      • ocharles
        Everything in my milestone that I can actually work is pretty minor, other than http://tickets.musicbrainz.org/browse/MBS-3017 which I still need to find more info on and http://tickets.musicbrainz.org/browse/MBS-2676 which needs me to finish off with fresh eyes tomorrow
      • warp
        sure, that doesn't make it any less serious :)
      • luks
        if we lived with it for 10 years, it can wait if there is more important work (which I'm sure there is)
      • ocharles: all ModBot errors fixed? :)
      • ocharles
        all known ones
      • not that it's too easy to filter a list of 700 tickets I have assigned to me :P
      • warp
        luks: ofcourse to some extent minor/critical is subjective. I have music my this particular artist in /mnt/music/tagged, so I'd more likely to be hit by it than most people :)
      • luks
        ocharles: go by the number of votes
      • ocharles
      • but I get told off for not working off my CI list :)
      • luks
        working on that search thing really seems like a waste of time to me
      • ocharles
        yep
      • luks
        or just adding to the list of exceptions, which currently contains only !!!
      • add
      • ocharles
        we only do that for indexed search atm, not direct searches
      • luks
        oh, I thought the direct search also used it
      • warp
        but it really should be much work to allow the direct search to do exact matches as well.
      • +not
      • ocharles: if you don't feel like working on it, assign it to me and I'll look into it.
      • ocharles
        i don't mind working on it, I just kinda agree that we have more pressing things
      • warp
        you cannot assign this artist to tracks and recordings, etc.. at all now. that seems pretty important.
      • kepstin-laptop joined the channel
      • bitmap joined the channel
      • ocharles
        right, time to wander home
      • ruaok joined the channel
      • voiceinsideyou1 joined the channel
      • ruaok: we're getting a lot of 502 errors the last few days
      • ruaok
        yes, I've been hearing that.
      • ocharles
        ah, ok
      • ruaok
        not sure what to do about it.
      • heard it on twitter, actually. ;)
      • ocharles
        yea, I saw that
      • but also saw it mentioned in IRC the day before
      • I thought it was due to having an open psql shell to totoro, but that's apparently not it
      • ruaok
        why would that do it?
      • ocharles
        connection count
      • ruaok
        the backend servers are all happy. nothing out of sorts when what I saw.
      • voiceinsideyou joined the channel
      • our connection count is nuts.
      • ocharles
        before I've accidently left a transaction open too, and that wrecked a bit of havoc
      • but that wasn't the problem this time (there was no transaction open)
      • ruaok
        *that* will do it.
      • but once you start hitting your connection limit, closing one connection won't help.
      • ocharles
        right
      • ruaok
        closing a 100 connections might have a positive effect.
      • ocharles
        it was only a wild guess, I doubt that's what it was
      • ruaok
        I think I will restart the front end proxy and see what that will do.
      • ocharles
        I think it was just coincidence 502s stopped briefly when I quit the shell
      • ruaok nods
      • reosarevokphone joined the channel
      • ruaok
        to diagnose 502s, here is your process:
      • 1. Are the back ends happy? if not, the front end requests are piling up for some reason. thus 502s.
      • warp pays attention.
      • ocharles
        how do I measure happiness?
      • tail the logs?
      • ruaok
        you look at server loads at the first clue.
      • ocharles
        ok
      • ruaok
        then you start looking at error/access logs
      • but the mrtg graphs are your first line of insight.
      • if the back ends are happy its likely that the proxy itself is having issues.
      • which is kinda hard to diagose at times.
      • but looking at the logs is the first thing to do.
      • ocharles
        other than just kick it and see what happens :)
      • warp
        ruaok: server load average should be < # of cores in the system?
      • ruaok
        ocharles: that used to be my preferred way of doing things, but rather ineffective. ;)
      • ocharles
        heh
      • ruaok
        warp: yeah.
      • given that, astro and pingu sitting at load 6 is cake.
      • totoro at 2... he's nearly napping he's so bored.
      • ocharles
        oh, talking of totoro, poke me when you're free later - need to discuss something with you
      • nikki
        totoro napping seems so fitting :P
      • ocharles
        haha
      • ruaok
        ocharles: after coffee. ;)
      • 2011/08/04 17:15:24 [warn] 26957#0: *1590920301 an upstream response is buffered to a temporary file
      • ocharles
        sure
      • ruaok
        we're seeing too many of those.
      • did we just push out something that made the size of the /ws/1/release responses bigger?
      • ok, I'll keep tailing the logs for 502s.
      • nikki prods ocharles