#musicbrainz-devel

/

      • ocharles
        we don't use RAWDATA anymore, and haven't for ages
      • git rev-parse HEAD
      • paste the output of that
      • Swarup
        8a9043141811e94314c78f522530fd68ff3f7d28
      • ocharles
        that's from July 2011
      • you need to: git fetch origin && git checkout v-2012-01-12-schema-change
      • but I don't think your REPLICATION_TYPE is RT_SLAVE either
      • which the blog post instructs
      • Swarup
        it is RT_SLAVE..not sure what u mean..
      • i had done those commands before but had some issues..let me do it again..
      • ocharles
        well it only runs "RunExport" if you're RT_MASTER, but if you're sure DBDefs is correct, then don't worry :)
      • Swarup
        oh u are correct..when i switched to default for now..it set to standalone..let me correct that..
      • is this good now..git rev-parse HEAD ee7b694698475b35b52e18486963830f62b85ed9
      • ./upgrade.sh Mon Jan 16 08:59:25 PST 2012 : Adding CAA flag to release_meta BEGIN CREATE TYPE ALTER TABLE COMMIT Mon Jan 16 08:59:30 PST 2012 : Adding support for finding edits by relationship type BEGIN ERROR: language "plperlu" does not exist HINT: Use CREATE LANGUAGE to load the language into the database. ROLLBACK Mon Jan 16 08:59:30 PST 2012 : Adding new edit indexes BEGIN ERROR: function extract_path_value(text, u
      • the_metalgamer joined the channel
      • let me do this step again..Install any necessary perl modules by running perl ./Makefile.PL and sudo make installdeps
      • FilipeTriworks joined the channel
      • ocharles
        sorry, checkout v-2012-01-12-schema-change-2
      • Swarup
        git checkout v-2012-01-12-schema-change-2 Previous HEAD position was ee7b694... Add a call to CREATE LANGUAGE in the CreatePLPerl init script, as this is important HEAD is now at eb89c2b... Another fix for LoadReplicationChanges and removed plperl updates for slaves in upgrade.sh
      • is that ok?
      • ocharles
        that's fine
      • upgrade.sh should not try and do that plperlu stuff now
      • Swarup
        should i do step 5 again before upgrade.sh
      • ocharles
        but I imagine it'll fail because part of the upgrade went through
      • nah
      • Swarup
        k
      • ./upgrade.sh Mon Jan 16 09:06:07 PST 2012 : Adding CAA flag to release_meta BEGIN ERROR: type "cover_art_presence" already exists ERROR: current transaction is aborted, commands ignored until end of transaction block ROLLBACK Mon Jan 16 09:06:07 PST 2012 : Going to schema sequence 14 UPDATE 1 Mon Jan 16 09:06:07 PST 2012 : UPDATE THE DB_SCHEMA_SEQUENCE IN DBDefs.pm TO 14 ! Mon Jan 16 09:06:07 PST 2012 : Done
      • is that good?
      • ocharles
        yes, I think you're done now
      • update the sequence to 14 in DBDefs
      • FilipeTriworks joined the channel
      • Swarup
        thanks..i need to try on the other VM host tomorrow..which is our actual master VM host..so actually the way we setup is ..our MB DBs are external and it's setup as Master DB and Replicate slave..so any changes done to Master DB would apply to our Slave DB..
      • marcooliveira joined the channel
      • ijabz
        resosarevok, hi
      • reosarevok: hi
      • ocharles
        ijabz: I set search server and mmd-schema up on ci btw, so nothing to do there :)
      • ijabz
        ocharles thumbs up
      • ocharles
        an xml schema change commit to mmd-schema will now retest both the search server and musicbrainz-server
      • ijabz
        ijabz has changed the topic to: Agenda: reviews, default to advanced search? (reosarevok), meta description for google (kepstin), Cover Art Proposal (ijabz)
      • thats usefu;l
      • reosarevok
        ijabz, hi
      • reosarevok is peeling potatoes
      • ijabz
        sounds fun
      • reosarevok
        So if I take a while to reply, that's it
      • ijabz
        I dint know if you wanted a pre chat about your advanced search agenda topic
      • reosarevok
        Can't hurt I guess
      • I am partially wondering why we have a non-advanced search at all, partially hoping I can at least get a preference for the search in the upper corner of MB to be advanced
      • hawke_
        +1
      • reosarevok
        As I've never seen a case in which I'd need to use the non-advanced search
      • ijabz
        And what do you consider the difference to be between advanced and not advanced
      • reosarevok
        As in, with advanced=1 activated
      • ijabz
        Yeah, sure but I'm interested to know what you think the difference is if you select/deselect that option
      • reosarevok
        I'd usually like to do searches involving advanced syntax
      • ijabz
        That is half of it.
      • reosarevok
      • ocharles
        warp: what calls /ws/js/freedb?
      • reosarevok
        The second result is much more useful, but getting to it involves either doing a search I don't need before
      • Or going to the search page to search there
      • ijabz
        The trouble is users who do not understand the lucene search syntax (which is most users) can end up creating invalid queries because they do not know that they have to escape some characters
      • reosarevok
        Then I guess a preference would make sense
      • I know what our devs think of preferences, but still
      • ocharles
        imo, it's not a case of advanced search here
      • but http://musicbrainz.org/search?query=planets+hol... should be the same as your advanced search
      • reosarevok
        Why?
      • ocharles
        because that's much more intuitive
      • if my dad wanted to find a specific release, he might try
      • reosarevok
        But that list also includes stuff that has both words on the title
      • ocharles
        "Pink floyd - dark side of the moon"
      • reosarevok
        Which makes sense
      • hawke_
        reosarevok: Hmm, one of those searches is for release group…
      • ocharles
        he doesn't want to know he has to type artist:"Pink Floyd" -- that'll confuse the hell out of him
      • ijabz
        The other half, is that we can massage the query entered by the user when not doing advanced search, and I want to do more of that
      • reosarevok
        hawke_, ooops, true
      • ocharles
        ijabz: that sounds like the better approach
      • reosarevok
        That would also make sense
      • Doesn't help my ability to do useful searches from there, though :p
      • ijabz
        So we could change the default search fields or modify the query that is entered when you do basic search
      • reosarevok also tends to want to do barcode and cat# searches, which again involve going to the search page first
      • ocharles
        pasting "FOOREC123" should search for catalog numbers
      • that should just work
      • ruaok joined the channel
      • ijabz
        We already do this for artist search so that it weights searches on artist, alias and sort name fields and weights pahrases over single matches
      • hawke_
        I’m still annoyed that the results are so completely different though, between basic and advanced.
      • ocharles
        it would seem like a non-advanced search should just search all indexes, and take whatever has the best score over all indexes
      • hawke_
        (as far as scoring)
      • kepstin-netbook
        ocharles: I've wanted that for years :/
      • hawke_
        ocharles: +1
      • MBChatLogger
      • kepstin-netbook
        either that, or allowing google to index mb.org so I could just use google search instead ;)
      • hawke_
        kepstin-netbook: That sounds like admitting failure. ;-)
      • ijabz
        actually hawk Ive decided to do implement some of your fuzzy matching ideas in the basic search http://tickets.musicbrainz.org/browse/SEARCH-167
      • hawke_
        ijabz: Thanks
      • ijabz
        But that doesn't mean you'll get the same results in basic and advanced, there would be no need for both if that was the case
      • hawke_
        ijabz: No, that’s fine. I just think that regular/basic search should give equally useful results for both
      • warp
        ocharles: hm, perhaps the freedb search in the "Add Disc" dialog of the relaese editor?
      • hawke_
        “aaron lebedeef” being the example there of course
      • ijabz
        ocharles:I think that should be a different option to allow search all indexes, and I did some work on that and opened a ticket on you to implement the guy bit
      • ocharles
        warp: nah, it's something that gives it a category and freedb id
      • ijabz
        s/guy/gui
      • ocharles
        ijabz: that was for all indexes and all types
      • reosarevok
        Ok
      • ocharles
        I mean a smaller change which is all indexes for a certain type
      • reosarevok
        Then
      • ocharles
        ijabz: at least, I thought you were looking at unified search, which is something a bit different
      • reosarevok
        reosarevok has changed the topic to: Agenda: reviews, preference for advanced search? (reosarevok), meta description for google (kepstin), Cover Art Proposal (ijabz)
      • :)
      • j-b
        thresh: 3 boxes are there... Consecutives IPs
      • ijabz
        Because I get annoyed when I search for a release or discos and I get artists and stuff as well
      • hawke_
        ijabz: I don’t expect the same result, I just don’t expect the stuff I’m looking for which seems obviously correct to be buried at the end of the list
      • ijabz: IMO that (type-specific search) should be a feature of advanced search
      • ijabz
        ocharles:yes i was, I don't understand the difference with 'non-advanced search should just search all indexes, and take whatever has the best score over all indexes' and that
      • ocharles
        ijabz: An artist search (non-advanced) should return only artist results, but it should search over all "artisty" indexes
      • so...
      • artist name, sort name, comment, aliases
      • warp
        ocharles: yes.
      • kepstin-netbook
        ijabz: ocharles' proposal will still require that you pick a type to search, e.g. "release" - it just means that you could enter "pink floyd CAT-###" and find a release.
      • ijabz
        hawk, IMO the point of the basic search is that if Im searching for an artist I can just enter an artist in the search field and it will find me the best matches in the usual case, I don't want it to return me releases or release-groups
      • ocharles
        and a release search should search all "releasey" indexes - release title, comment, artist credits, etc
      • warp
        ocharles: the add disc dialog does a search on /ws/js/freedb?q=ALBUMTITLE&artist=ARTIST etc..
      • ocharles
        warp: oh, but the action says Args(2)
      • warp
        ocharles: if a search result is expanded it calls e.g. http://beta.musicbrainz.org/ws/js/freedb/blues/...
      • ocharles
        ah
      • thanks!
      • ijabz
        Whereas the advanced search allows me to search the artist index based on any criteria I choose, depending on what Im trying to achieve
      • ocharles
        advanced search is fine how it is, imo
      • ijabz
        ocharles:basic search does do that, but maybe needs to add a few more fields to search on
      • ocharles
        ok, sounds like it just needs more tuning then
      • ijabz
        Actually, you know currently you have some code on the musicbrainz server (I provided the regexp) that rewrites the artist query it sends to the search server, this query rewrite should be done on the search server
      • ocharles
        +1
      • I was going to say that
      • :)
      • ijabz
        Then I can do it more efficiently and much more easily improve the results returned by basic search
      • ocharles nods
      • ocharles
        it's also not musicbrainz-server's job
      • ijabz
        Great, well I will add an extra parameter such as 'basic=true' and then you can just pas me the user entered value as is, and I wil deal with it
      • ocharles
        ?advanced=1 would match better I think
      • ijabz
        meaning ?
      • ocharles
        meaning "this is an advanced search"
      • rather than being explicit about basic searches, i'd rather be explicit about advanced searches
      • because if our search is doing its job, advanced search should be the exception :)
      • ijabz
        But advanced is the default, i.e when using the web service it uses advanced
      • though I would like to add the option to use basic version from the web service as well
      • ocharles
        that feels inconsistent
      • ocharles nods
      • but I can just pass advanced=1 from web service queries to the search server
      • j-b
        thresh: 88.190.34.17|18|19
      • ijabz
        There is certainly a problem with the name, but what do you mean inconsistent
      • ocharles
        it's inconsistent that the default search on the website is basic, and advanced in the web service
      • but that's tangential to this discussion :)