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
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
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..
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