we don't use RAWDATA anymore, and haven't for ages
2012-01-16 01646, 2012
ocharles
git rev-parse HEAD
2012-01-16 01650, 2012
ocharles
paste the output of that
2012-01-16 01614, 2012
Swarup
8a9043141811e94314c78f522530fd68ff3f7d28
2012-01-16 01636, 2012
ocharles
that's from July 2011
2012-01-16 01610, 2012
ocharles
you need to: git fetch origin && git checkout v-2012-01-12-schema-change
2012-01-16 01626, 2012
ocharles
but I don't think your REPLICATION_TYPE is RT_SLAVE either
2012-01-16 01631, 2012
ocharles
which the blog post instructs
2012-01-16 01649, 2012
Swarup
it is RT_SLAVE..not sure what u mean..
2012-01-16 01626, 2012
Swarup
i had done those commands before but had some issues..let me do it again..
2012-01-16 01654, 2012
ocharles
well it only runs "RunExport" if you're RT_MASTER, but if you're sure DBDefs is correct, then don't worry :)
2012-01-16 01621, 2012
Swarup
oh u are correct..when i switched to default for now..it set to standalone..let me correct that..
2012-01-16 01658, 2012
Swarup
is this good now..git rev-parse HEAD ee7b694698475b35b52e18486963830f62b85ed9
2012-01-16 01638, 2012
Swarup
./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
2012-01-16 01608, 2012
the_metalgamer joined the channel
2012-01-16 01636, 2012
Swarup
let me do this step again..Install any necessary perl modules by running perl ./Makefile.PL and sudo make installdeps
2012-01-16 01642, 2012
FilipeTriworks joined the channel
2012-01-16 01612, 2012
ocharles
sorry, checkout v-2012-01-12-schema-change-2
2012-01-16 01600, 2012
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
2012-01-16 01612, 2012
Swarup
is that ok?
2012-01-16 01615, 2012
ocharles
that's fine
2012-01-16 01623, 2012
ocharles
upgrade.sh should not try and do that plperlu stuff now
2012-01-16 01639, 2012
Swarup
should i do step 5 again before upgrade.sh
2012-01-16 01639, 2012
ocharles
but I imagine it'll fail because part of the upgrade went through
2012-01-16 01648, 2012
ocharles
nah
2012-01-16 01659, 2012
Swarup
k
2012-01-16 01625, 2012
Swarup
./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
2012-01-16 01647, 2012
Swarup
is that good?
2012-01-16 01655, 2012
ocharles
yes, I think you're done now
2012-01-16 01659, 2012
ocharles
update the sequence to 14 in DBDefs
2012-01-16 01659, 2012
FilipeTriworks joined the channel
2012-01-16 01624, 2012
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..
2012-01-16 01610, 2012
marcooliveira joined the channel
2012-01-16 01622, 2012
ijabz
resosarevok, hi
2012-01-16 01627, 2012
ijabz
reosarevok: hi
2012-01-16 01649, 2012
ocharles
ijabz: I set search server and mmd-schema up on ci btw, so nothing to do there :)
2012-01-16 01610, 2012
ijabz
ocharles thumbs up
2012-01-16 01658, 2012
ocharles
an xml schema change commit to mmd-schema will now retest both the search server and musicbrainz-server
2012-01-16 01651, 2012
ijabz
ijabz has changed the topic to: Agenda: reviews, default to advanced search? (reosarevok), meta description for google (kepstin), Cover Art Proposal (ijabz)
2012-01-16 01618, 2012
ijabz
thats usefu;l
2012-01-16 01649, 2012
reosarevok
ijabz, hi
2012-01-16 01654, 2012
reosarevok is peeling potatoes
2012-01-16 01601, 2012
ijabz
sounds fun
2012-01-16 01603, 2012
reosarevok
So if I take a while to reply, that's it
2012-01-16 01632, 2012
ijabz
I dint know if you wanted a pre chat about your advanced search agenda topic
2012-01-16 01616, 2012
reosarevok
Can't hurt I guess
2012-01-16 01651, 2012
reosarevok
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
2012-01-16 01617, 2012
hawke_
+1
2012-01-16 01623, 2012
reosarevok
As I've never seen a case in which I'd need to use the non-advanced search
2012-01-16 01625, 2012
ijabz
And what do you consider the difference to be between advanced and not advanced
2012-01-16 01636, 2012
reosarevok
As in, with advanced=1 activated
2012-01-16 01603, 2012
ijabz
Yeah, sure but I'm interested to know what you think the difference is if you select/deselect that option
2012-01-16 01604, 2012
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
2012-01-16 01633, 2012
reosarevok
Or going to the search page to search there
2012-01-16 01633, 2012
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
2012-01-16 01658, 2012
reosarevok
Then I guess a preference would make sense
2012-01-16 01610, 2012
reosarevok
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
2012-01-16 01612, 2012
hawke_
ijabz: No, that’s fine. I just think that regular/basic search should give equally useful results for both
2012-01-16 01620, 2012
warp
ocharles: hm, perhaps the freedb search in the "Add Disc" dialog of the relaese editor?
2012-01-16 01628, 2012
hawke_
“aaron lebedeef” being the example there of course
2012-01-16 01628, 2012
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
2012-01-16 01636, 2012
ocharles
warp: nah, it's something that gives it a category and freedb id
2012-01-16 01636, 2012
ijabz
s/guy/gui
2012-01-16 01646, 2012
ocharles
ijabz: that was for all indexes and all types
2012-01-16 01655, 2012
reosarevok
Ok
2012-01-16 01655, 2012
ocharles
I mean a smaller change which is all indexes for a certain type
2012-01-16 01657, 2012
reosarevok
Then
2012-01-16 01607, 2012
ocharles
ijabz: at least, I thought you were looking at unified search, which is something a bit different
2012-01-16 01610, 2012
reosarevok
reosarevok has changed the topic to: Agenda: reviews, preference for advanced search? (reosarevok), meta description for google (kepstin), Cover Art Proposal (ijabz)
2012-01-16 01616, 2012
reosarevok
:)
2012-01-16 01617, 2012
j-b
thresh: 3 boxes are there... Consecutives IPs
2012-01-16 01624, 2012
ijabz
Because I get annoyed when I search for a release or discos and I get artists and stuff as well
2012-01-16 01650, 2012
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
2012-01-16 01615, 2012
hawke_
ijabz: IMO that (type-specific search) should be a feature of advanced search
2012-01-16 01642, 2012
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
2012-01-16 01639, 2012
ocharles
ijabz: An artist search (non-advanced) should return only artist results, but it should search over all "artisty" indexes
2012-01-16 01640, 2012
ocharles
so...
2012-01-16 01646, 2012
ocharles
artist name, sort name, comment, aliases
2012-01-16 01651, 2012
warp
ocharles: yes.
2012-01-16 01657, 2012
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.
2012-01-16 01612, 2012
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
2012-01-16 01623, 2012
ocharles
and a release search should search all "releasey" indexes - release title, comment, artist credits, etc
2012-01-16 01628, 2012
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
2012-01-16 01641, 2012
ocharles
advanced search is fine how it is, imo
2012-01-16 01609, 2012
ijabz
ocharles:basic search does do that, but maybe needs to add a few more fields to search on
2012-01-16 01611, 2012
ocharles
ok, sounds like it just needs more tuning then
2012-01-16 01642, 2012
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
2012-01-16 01657, 2012
ocharles
+1
2012-01-16 01600, 2012
ocharles
I was going to say that
2012-01-16 01603, 2012
ocharles
:)
2012-01-16 01612, 2012
ijabz
Then I can do it more efficiently and much more easily improve the results returned by basic search
2012-01-16 01620, 2012
ocharles nods
2012-01-16 01625, 2012
ocharles
it's also not musicbrainz-server's job
2012-01-16 01657, 2012
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
2012-01-16 01622, 2012
ocharles
?advanced=1 would match better I think
2012-01-16 01643, 2012
ijabz
meaning ?
2012-01-16 01657, 2012
ocharles
meaning "this is an advanced search"
2012-01-16 01608, 2012
ocharles
rather than being explicit about basic searches, i'd rather be explicit about advanced searches
2012-01-16 01620, 2012
ocharles
because if our search is doing its job, advanced search should be the exception :)
2012-01-16 01641, 2012
ijabz
But advanced is the default, i.e when using the web service it uses advanced
2012-01-16 01656, 2012
ijabz
though I would like to add the option to use basic version from the web service as well
2012-01-16 01657, 2012
ocharles
that feels inconsistent
2012-01-16 01602, 2012
ocharles nods
2012-01-16 01631, 2012
ocharles
but I can just pass advanced=1 from web service queries to the search server
2012-01-16 01639, 2012
j-b
thresh: 88.190.34.17|18|19
2012-01-16 01654, 2012
ijabz
There is certainly a problem with the name, but what do you mean inconsistent
2012-01-16 01619, 2012
ocharles
it's inconsistent that the default search on the website is basic, and advanced in the web service
