09:33 |
bshum |
dbwells: While we're not using a fully updated master in our production (we need to update to get the new title browse stuff), we do have the beginnings of browse in play and haven't seen any extremely long browse attempts yet. |
09:35 |
kmlussier |
bshum: I think it might be something that was just recently introduced because the browse queries had previously been working on Dyrcona's server. I believe it's just in the past few weeks that they became problematic. |
09:36 |
dbwells |
bshum: for our production data set, it seemed to be every query. I suspect it has to do with 0837 on large DBs. |
09:36 |
bshum |
Alternatively, is this something related to auth records being linked to browse? In which case, I couldn't properly test it anyways. |
09:36 |
bshum |
And yeah, 0837, gotcha..... |
09:37 |
dbwells |
bshum: Do you have 0837 in place? We haven't done any authority linking yet at all. |
09:37 |
dbwells |
also, test branch is moments away... |
09:38 |
bshum |
Aha, 0837 is not on production yet. |
09:38 |
bshum |
It's on my test server which is awaiting finishing touches |
09:38 |
bshum |
So I guess I'll test it there |
09:38 |
bshum |
dbwells++ # I'll let you know what I find out as well later today, if Dyrcona doesn't beat me to it. |
09:40 |
dbwells |
bshum: thanks! |
09:40 |
csharp |
browse is broken on our pines test server too: http://next.gapines.org |
09:41 |
csharp |
I was starting with the assumption that our cold cache and slow DB server is the cause |
09:41 |
kmlussier |
I don't know if Dyrcona changed something this morning, but the browse search on his server is happier today than it has been in a long time. |
09:41 |
csharp |
(that's running 2.5-rc1, btw) |
09:42 |
Dyrcona |
I applied the changes from lp 1244432 last night when I updated the code to fix a typo in one of my development branches. |
11:25 |
doug_ |
jeff: thanks, that does help a lot, and contains a lot of what I'm looking for. |
11:26 |
Dyrcona |
dbwells: Reaction. |
11:29 |
jeff |
"I just wish we could get course credit for this." |
11:31 |
Dyrcona |
dbwells / kmlussier: I'll apply the complete branch on lp 1244432 tonight for testing tomorrow. I want to reload my database to test a script for deleting tons of copies again tomorrow. |
11:31 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
11:32 |
bshum |
jeff: Sounds like Marty to me. |
11:33 |
bshum |
:) |
12:49 |
rfrasur |
Dyrcona: I only have experience with C ++ and it was only a semester...so enough to say I learned something for a month or so...and then never use again. |
12:49 |
rfrasur |
(it was a fun elective though) |
12:50 |
|
Sato`kun joined #evergreen |
12:52 |
bshum |
dbwells: Tested browse with my test server and production data and you were right that it was pretty broken. |
12:52 |
bshum |
Applied those alter functions and things went smoothly after that. |
12:52 |
csharp |
csharp## |
12:53 |
bshum |
I'll do some more testing after lunch. |
12:53 |
rfrasur |
lol, csharp++ |
11:21 |
jeff |
in your case, i think the migration and post migration cleanup may have gotten it out of sync. |
11:21 |
jeff |
eeevil: liar! :-) |
11:22 |
eeevil |
lines 215-260, just replace the CREATE TABLE with a TRUNCATE |
11:23 |
jeff |
Bmagic: as eeevil indicated, the source sql file 080.schema.money.sql should give you the info you need to rebuild that materialized view -- but i'd recommend caution. test on a non-production copy of your database, and read / understand / ask questions here until you're comfortable with doing it. |
11:24 |
* dbs |
swore there was a "refresh materialized billable xact summary" function, too |
11:25 |
eeevil |
dbs: I couldn't find it, but, yeah, I thought there was one |
11:26 |
Bmagic |
jeff: Right on. I'm working with test db right now. Just so I understand, the operation would be to delete all rows inside of the mmbxs , the recreate with the 080.schema.money.sql ? |
11:26 |
eeevil |
dbs: I bet we're thinking of the "simple record extracts" matview |
11:26 |
jeff |
I think it used to be "truncate mmbxs and insert from mbxs", but for a while now mbxs has just been a pointer to mmbxs. |
11:27 |
jeff |
Bmagic: essentially, but you can use TRUNCATE on the mmbxs table, and you don't need all of that sql file to re-create things. |
13:30 |
* rfrasur |
opts for the second idea because I'm still a bit of a chicken. |
13:32 |
Dyrcona |
gmcharlt++ # For maintaining CPAN packages related to MARC |
13:32 |
jcamins |
rfrasur: ouch. |
13:34 |
dbs |
Am I crazy or is the current set of pgTAP tests basically fetching a list of objects from the PostgreSQL info schema, then iterating over that list to verify that those objects exist? |
13:36 |
dbs |
Seems like it is a test of PostgreSQL's info schema, with very little Evergreen-specific testing |
13:37 |
gmcharlt |
dbs: I believe the idea is that you would run make-pgtap-tests.sql on a known-clean database, then use them to test a production database for schema variances |
13:37 |
gmcharlt |
where "them" == the generated tests |
13:38 |
jeff |
oh. that sounds useful. i might look at that. |
13:38 |
dbs |
gmcharlt: oh. are there any separate pgTAP tests for Evergreen-specific functions? |
13:39 |
* dbs |
has just added one for ingesting a record and checking the md5 hash of the MARC to ensure the processing has proceeded as expected |
13:39 |
gmcharlt |
dbs: a few -- see Open-ILS/src/sql/Pg/t |
13:40 |
dbs |
gmcharlt: ah, the make-pgtap-tests.pl in sql/Pg/ threw me off |
13:41 |
gmcharlt |
make-pgtab-schema-tests.pl might be a better name |
13:42 |
dbs |
Might be useful to add some comments about the intended usage into make-whatever-tests.pl too |
13:46 |
dbs |
might add a pointer to "pg_prove" in t/README too. This is good stuff |
13:52 |
jboyer-isl |
rfrasur: You had your gadget discussion while I was at lunch, but I wanted to throw a couple cents at it. JCPL picked up a few gadgets (similar to the ISL kit, but hand-picked by me) with things like every type of available kindle, a chromebook, ipod, ipad, galaxy tab, and some nooks |
13:52 |
jboyer-isl |
Staff could check out devices to learn on, or to show patrons how to use Overdrive, ebrary, etc. |
13:53 |
jboyer-isl |
they also made an appearance at a staff day, as a kind of tech petting zoo. |
15:21 |
dbs |
/win/win 11 |
15:21 |
dbs |
meh |
15:21 |
dbs |
/lose/lose :) |
15:26 |
Bmagic |
jeff: LOL - it's the row that I fixed as a test |
15:27 |
jeff |
aha. :-) |
15:27 |
jeff |
which query did it show in? |
15:28 |
Bmagic |
I used a "not in" strategy there |
09:35 |
dbs |
You could create an actual source for "source unknown" and link it up and enforce the constraint, but I don't see how much that buys us |
09:35 |
Dyrcona |
Well, the source is usually known, someone just neglects to set it. |
09:36 |
Dyrcona |
makes queries fun: and (source is null or source in (1,2)) |
09:36 |
dbs |
bshum: Do you have an example RDA record you could supply for testing purposes? I think the fix may just be to add another get_graphic_880s() call right after the one for 260, similar to what we do for authors |
09:36 |
tsbere |
How about "we know, but the source isn't in the list"? Because I suspect sometimes that happens. ;) |
09:36 |
Dyrcona |
tsbere: I don't suspect that happens very often with our gang, and in that case, they should open a ticket to request the source be added. :) |
09:37 |
dbs |
Dyrcona: about 2 million of our bibs would be "source = migrated from previous system"; nobody is going to spend time tracking down the actual source for all of those unless there is some benefit |
10:01 |
bshum |
dbs: As for getting some sample RDA bibs, I could see that being helpful to include in our sample datasets for the future. I'll see what I can do on that front as well. |
10:01 |
jeff |
of course, some OU settings make no sense to have a workstation or usr level counterpart, so maybe not. |
10:02 |
bshum |
(though probably later) |
10:03 |
jeff |
while i have high hopes to test some more 2.5 goodness this evening or weekend, today i am fixing bits and adding bits in the user editor. |
10:03 |
bshum |
2.3 is unaffected since that was pre-880 stuff |
10:03 |
jeff |
one of the things it has me thinking about is how you'd specify an OU setting for "this is how you generate a suggested username" |
10:04 |
|
yboston joined #evergreen |
16:58 |
rfrasur |
oh, I don't kick them out. I just make them behave. They can't cover their faces and have to have on clothes over the suits. |
16:58 |
rfrasur |
There's also a kid with a pigeon mask. |
16:59 |
rfrasur |
It was kinda impressive...but yeah. |
16:59 |
anton____ |
hello everybody, I am testing the holds module in EG and I can´t get the Pull list, I mean, I hold a book with one test account, and then I try to see this hold in Pull List for hold requests but It doesn´t show nothing |
17:00 |
Dyrcona |
anton____: You have to run the hold targeter to target the hold at a copy before it will show up in the pull list. |
17:01 |
anton____ |
Is in EG site a complete documentation about how do that= |
17:01 |
anton____ |
? |
09:15 |
|
jbfink joined #evergreen |
09:16 |
|
mrpeters joined #evergreen |
09:24 |
Dyrcona |
dbs: Is there a solution that works whether one has upgraded or not? |
09:27 |
dbs |
Dyrcona: It would be great if someone not on one of those distros would test my branch and tell me how much breaks :) |
09:30 |
Dyrcona |
dbs: I only look at LTS releases from Ubuntu, so 14.04 would be the next that I experiment with. |
09:31 |
dbs |
Dyrcona: right, but if you have a 12.04 box at the ready it would be instructive to just check out the branch and run 'make check' on it |
09:32 |
Dyrcona |
dbs: Will do. |
10:01 |
dbs |
Dyrcona: does make check |
10:01 |
dbs |
Dyrcona: does "make check" pass for starters? |
10:01 |
dbs |
And then if that passes, "eg_db_config <yada yada> --load-all-sample"? |
10:02 |
Dyrcona |
I'm testing this on a system that already has data loaded. |
10:02 |
dbs |
three queries from the 12th? Wow. |
10:02 |
paxed |
dbs: not sure. i know i haven't done such a post. i could try to compile one... |
10:03 |
dbs |
Dyrcona: ah, okay, my branch doesn't have any upgrade scripts for the database functions yet, so just "make check" or running "prove -l lib t" in the Open-ILS/src/perlmods directory would be great |
10:03 |
Dyrcona |
I'm about to paste the output from make.check, looks some tests fail. |
10:04 |
dbs |
paxed: yeah. I mean, I know you have a long list of bugs but some level of abstraction & prioritization would probably help us understand our failings |
10:04 |
dbs |
Dyrcona: thanks |
10:05 |
pastebot |
"Dyrcona" at 64.57.241.14 pasted "make check > ../make.out 2>&1" (131 lines) at http://paste.evergreen-ils.org/28 |
10:05 |
Dyrcona |
One of the tests that fails is entityize diacritics. |
10:05 |
dbs |
Dyrcona: perfect. that's what I was worried about. |
10:05 |
paxed |
dbs: for example, today i found out the "Serials" as done by both Eg and Koha does not really serve our needs - our expert on it said the Serials is more for scientific papers and such, rather than "periodicals" (as he said), or magazines. |
10:06 |
Dyrcona |
paxed: You'll just have to implement your own ILS, then. :) |
10:12 |
paxed |
dbs: well, then it could be something that's missing from our database, and clicking on one of the issues under "issues held" does nothing. |
10:12 |
paxed |
s/and/as/ |
10:12 |
paxed |
(in opac) |
10:12 |
dbs |
Dyrcona: oh, fascinating. "prove -l lib t/14-OpenILS-Utils.t" works fine on its own, which is where I was focused on fixing the problem that had popped up, but last night / this morning I failed to run all the tests |
10:12 |
jboyer-isl |
paxed, Dyrcona: Somebody beat you to it: http://en.wikipedia.org/wiki/NewGenLib |
10:13 |
dbs |
jboyer-isl: oh yeah, that's been around a long time |
10:13 |
jboyer-isl |
Even has Acq. I tried to set it up in 2008-ish. Daunting is the polite term. |
10:53 |
bshum |
Or on the weekend. |
10:54 |
jcamins |
Heh. What time did you get home? |
10:54 |
bshum |
Around 1:40 or so |
10:55 |
jcamins |
Eww. I took a cab and was back before 12:30. |
10:56 |
jcamins |
Usually I am frugal and take the bus/subway, but I just couldn't bring myself to test the late night schedule. |
10:56 |
bshum |
Indeed, it was late. |
10:57 |
bshum |
Good to be home though. |
10:57 |
|
ktomita joined #evergreen |
12:35 |
bshum |
I'll try it again once I get a new fresh system going later this week. |
12:37 |
csharp |
yeah - I was going to experiment with reverting 3335c54 just to see if that makes a difference - nothing about that commit touches 'obj.active_services' though, so dunno ;-) |
12:37 |
pinesol_green |
[evergreen|Bill Erickson] Import bib trash fields : XUL Z39.50 UI - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=3335c54> |
12:41 |
jeff |
ah yes... that lovely moment when you're testing your user editor changes and the username is replaced with: tfunctionanonymousnifargumentslengththisanreturnthisa |
12:49 |
gmcharlt |
heh |
12:54 |
jeff |
also fun: those moments where you spend some much time trying to write a non-awkward comment that you also end up re-factoring the one if statement that you were commenting |
12:54 |
jeff |
...not in a significant way, but just so that the ordering matches the ordering of the comment, because you have no interest in re-writing the comment Yet Again. |
14:26 |
|
BigRig__ joined #evergreen |
14:30 |
|
BigRig joined #evergreen |
14:30 |
JennB |
hello Evergreen Community, can you tell me if EG can be in Arabic? I saw this listserv message http://list.georgialibraries.org/pipermail/open-ils-general/2011-March/004343.html but as you see its from 2011 |
14:31 |
* bshum |
imagines it's possible, but doubts we've tested RTL extensively |
14:31 |
bshum |
(or at all) |
14:31 |
JennB |
ok, thanks Ben! |
14:32 |
bshum |
In theory, if we had the translations for arabic, we could try reviewing various parts of the GUI for right to left compatibility. |
14:32 |
Dyrcona |
Well, I've processed some records recently that had Arabic in them. |
17:18 |
tsbere |
hopkinsju: An attempt to say "the user is physically at...." though we use it with subdomains to say "this subdomain represents..." |
17:18 |
tsbere |
I believe it can be set with a url param in addition to IP based and apache-level configs |
17:19 |
hopkinsju |
tsbere: Any chance of documentation on this? Particularly the apache config |
17:20 |
tsbere |
hopkinsju: At the apache level you set an environment variable named "physical_loc"...for that matter for your testing just add physical_loc=id to the query string (add a ? or & or whatever before it as needed) |
17:20 |
hopkinsju |
Gotcha |
17:21 |
tsbere |
we do some fancy rewrite tricks to set all of that stuff for our subdomains, but you can also just hardcode it if you have widely varying configs |
17:22 |
tsbere |
hopkinsju: Once you have physical_loc set then the hiding depth (if >1) says "start the tree at that depth". 1 would be the "System" in a standard setup, I believe (with 0 being "everything" and 2 being "branch within the system") |
09:42 |
|
yboston joined #evergreen |
09:42 |
|
kmlussier joined #evergreen |
09:42 |
RoganH |
Patience is more of a necessity than a virtue with Overdrive. |
09:42 |
jeff |
They claim to review applications on a "weekly basis", but we applied Oct 1 and it is now Oct 15. |
09:43 |
jeff |
At this rate, I could have driven down to Cleveland, spent some time with friends, and knocked on OverDrive's door for a personal interview to obtain access to the Circulation APIs and/or test environment. ;-) |
09:43 |
RoganH |
Time is highly subjective. |
09:44 |
RoganH |
They may only be counting the third hour of every meal break towards the week. |
09:48 |
|
misilot joined #evergreen |
12:08 |
pinesol_green |
Launchpad bug 1239837 in Evergreen 2.4 "Undefined variable prevents quick PO creation" (affected: 1, heat: 6) [High,New] https://launchpad.net/bugs/1239837 |
12:10 |
berick |
dbwells: looking now, sorry |
12:10 |
dbwells |
np |
12:13 |
berick |
dbwells++ looks correct, testing and signing off now |
12:14 |
dbwells |
berick++ # thank you, sir |
12:34 |
gmcharlt |
berick++ |
12:34 |
berick |
does this look odd to anyone else? http://dev198.esilibrary.com/eg/opac/results?query=violin&qtype=keyword |
13:21 |
gmcharlt |
#info gmcharlt to release 2.2.1 this week |
13:22 |
kmlussier |
gmcharlt++ |
13:22 |
gmcharlt |
(and KohaCon as a vacation for Koha's release manager? you jest, surely! ;) ) |
13:22 |
kmlussier |
#info csharp to test 2.4.2 rc so we can officialize it |
13:22 |
eeevil |
gmcharlt: this includes the new opensrf controller script from berick, yes? |
13:22 |
bshum |
Related to opensrf, is there anything in master that lends cutting of a 2.3.0? |
13:22 |
eeevil |
kmlussier: that happened, 2.4.3 is waiting in the wings |
13:27 |
csharp |
senator++ |
13:27 |
gmcharlt |
as recommended, with note that 2.2.1 is compatible |
13:27 |
eeevil |
(yes, 2.3+ recommended) |
13:28 |
kmlussier |
#info 2.3.0 will be recommended to run with EG 2.5, but 2.2.1 will be compatible as well. |
13:28 |
kmlussier |
Moving on to the next action item... |
13:28 |
kmlussier |
#info senator to find old incomplete action items to move forward (with help from the channel in making sure I get the right ones) - Done! |
13:29 |
kmlussier |
senator++ # Thanks for all the action items! |
13:29 |
kmlussier |
#info eeevil to commit a techref doc explaining how to build pgTAP test |
13:31 |
eeevil |
kmlussier: I have not done that yet. it's on my gtasks list, though! |
13:31 |
eeevil |
nor have I had time to do the next one |
13:31 |
kmlussier |
OK, I'll add both as action items for next time then. |
13:31 |
kmlussier |
#action eeevil to commit a techref doc explaining how to build pgTAP test |
13:32 |
eeevil |
thanks |
13:32 |
kmlussier |
#action o publish detailed plan about freezing baseline schemas between EG releases and using deprecates/supersedes in database upgrade scripts. This will go on the mailing list and the thread should structure further discussion of pros and cons of eeevil's plan. |
13:32 |
kmlussier |
That's it for action items, and I think we've also covered OpenSRF Release Info? |
13:33 |
kmlussier |
#topic Evergreen release info |
13:33 |
|
Callender_ joined #evergreen |
13:33 |
kmlussier |
dbwells: Should we start with 2.5? |
13:34 |
dbwells |
Sure. After a couple brief rounds of testing and bug squashing, RC1 is uploaded to the usual. |
13:34 |
dbwells |
usual place, that is |
13:35 |
dbwells |
Been having fun working out the release notes, but that is done as well, and will be uploaded shortly. |
13:35 |
|
Callender__ joined #evergreen |
13:36 |
kmlussier |
#info 2.5 RC1 is uploaded. Release notes will be uploaded shortly. |
13:36 |
kmlussier |
dbwells: Anything else to add? |
13:36 |
dbwells |
Since we are a day late already for the delayed version of 2.5 final, I am not sure where to pin down the timeline, but I am pretty confident things we'll get there soon. |
13:37 |
dbwells |
It's really all about more testing at this point, so maybe RC will inspire some folks. |
13:37 |
|
Callender joined #evergreen |
13:37 |
dbwells |
That's about it, unless folks have questions I might be able answer. |
13:37 |
kmlussier |
How much time do we usually allow for testing between RC1 and final? |
13:38 |
* csharp |
is testing beta1, but will upgrade to RC |
13:38 |
gmcharlt |
dbwells: apologies if it's already been mentioned, but I'd like to take this opportunity to mention the fixes for bug 1086458 |
13:38 |
pinesol_green |
Launchpad bug 1086458 in Evergreen 2.4 "Staff client memory leaks in 2.3 and later" (affected: 11, heat: 76) [High,Confirmed] https://launchpad.net/bugs/1086458 |
13:39 |
gmcharlt |
whether or not it's a candidate for inclusion in 2.5.0, I'm pretty sure lots of folks will be very happy if it makes it in by 2.5.1 |
13:39 |
dbwells |
gmcharlt: I haven't looked at it, but I certainly will. |
13:40 |
kmlussier |
I think a lot of people would love to see those patches in sooner rather than later. I'm not sure how to approach testing, though. |
13:41 |
csharp |
kmlussier: our approach will probably be a due diligence approach on a test server to make sure nothing breaks, then move to production as soon as we're satisfied |
13:41 |
dbwells |
gmcharlt: It doesn't look too bad, and if there are no objections, I can pull back RC1 and cram it in there. No formal announcements have been made yet. |
13:41 |
csharp |
seems like it's too difficult to simulate in a test environment |
13:41 |
kmlussier |
csharp++ |
13:42 |
csharp |
dbwells: +1 # fwiw |
13:42 |
eeevil |
dbwells: I know it's a pain, having just gone through packaging, but I'm +1 to that |
13:43 |
gmcharlt |
dbwells: thanks -- but also, TIA to anybody who tests it |
13:43 |
kmlussier |
Is everyone in agreement to include the memory leak patches in RC1 then? |
13:43 |
* gmcharlt |
will search for a suitable libation whose name encodes the concept of "at the last minute" for delivery next time I see you ;) |
13:44 |
dbwells |
I would also appreciate a test/sign-off/push to master on that branch by anyone else who can take a look. |
13:44 |
gmcharlt |
+1 |
13:45 |
* dbwells |
will test what he can of it, but RC1 will be out today, or BUST |
13:45 |
csharp |
dbwells++ |
13:46 |
kmlussier |
#agree dbwells will pull back RC1 and incorporate memory leak patches. Sign-off from other testers will be appreciated. |
13:46 |
kmlussier |
Any other questions for dbwells? |
13:52 |
jsime |
e.g. a patron search which could use the flesh feature to grab both the search results' list of patrons and those patrons' details |
13:52 |
jsime |
instead of getting the list of patrons, then iterating through it to get each of their details in turn |
13:56 |
kmlussier |
I think one thing we were thinking we might do with that information is include it as part of some kind of best practices for building a new web client. Or maybe there is a better way to share that information as we proceed with the analysis? |
13:56 |
phasefx |
jsime: just want to make sure you've seen this: http://wiki.evergreen-ils.org/doku.php?id=dev:testing:performance_issues |
13:57 |
jsime |
phasefx: thanks for the link - berick's notes under General Improvement are spot on for that |
13:58 |
kmlussier |
OK, we're running short on time. I'm going to move on to the next agenda item. |
13:58 |
kmlussier |
#topic Discuss freezing of master during beta/RC phase |
18:50 |
|
kbeswick joined #evergreen |
18:58 |
|
kbeswick joined #evergreen |
19:02 |
|
hopkinsju joined #evergreen |
19:13 |
dbwells |
After much gnashing of teeth, I finally have updated RC1 bits in the usual places. Please feel free to grab them and test if you are so inclined. |
19:15 |
dbwells |
I haven't been able to update the downloads page, as it is currently locked by another editor. At this point, I guess that part will have to wait for tomorrow. |
19:16 |
* dbwells |
head for home |
19:18 |
* dbwells |
listens to that command to himself :) |
19:46 |
|
kbeswick joined #evergreen |
19:56 |
|
j_scott joined #evergreen |
20:34 |
|
stevenyvr2 left #evergreen |
14:41 |
rfrasur |
but maybe not |
14:41 |
* rfrasur |
hasn't done much using the bib source |
14:41 |
RoganH |
I'll be honest, I can use the reports interface but I have to think about it. I mostly use sql. |
14:41 |
gmcharlt |
rfrasur: then there's the scenario where online circulation works perfectly when the main catalog is offline -- due to somebody not noticing that they were connecting to the test system |
14:42 |
rfrasur |
gmcharlt: please never tell my staff there's more than one server. |
14:42 |
* rfrasur |
has visions of mushroom clouds |
14:43 |
RoganH |
rfrasur: so .... no staff development day discussion of load balancing and bricks? |
14:43 |
gmcharlt |
rfrasur: let's just say that we now will consider turning off a test system temporarily at appropriate times to avoid asploding heads |
14:43 |
RoganH |
rfrasur: we could explain round robin database setups to them :) |
14:43 |
rfrasur |
no....not this staff. They're great, but technology is perhaps not their greatest area of interest and expertise. |
14:44 |
rfrasur |
gmcharlt++ |
15:38 |
senator |
i remember fixing this once in a totally different, not-serials interface |
15:39 |
senator |
i had though the way i did that should apply to cases like this too, but will check it out |
15:39 |
senator |
oh, well you have a solution already too |
15:40 |
jeff |
Hrm. fieldmapper.standardRequest seems to be able to get into a state where none of oncomplete, onerror, ontransporterror, nor onmethoderror will fire. |
15:41 |
jeff |
Testing in both xulrunner and Firefox |
15:42 |
jeff |
My test is to force a user edit save to fail by creating a usrname collision. |
15:42 |
jeff |
(which isn't handled in register.js, and I was trying to add handling) |
15:44 |
senator |
jeff: so the error is a DATABASE_QUERY_FAILED event? |
15:44 |
|
dMiller joined #evergreen |
15:45 |
senator |
i would expect oncomplete to fire for that |
16:11 |
rfrasur |
yep...and that was a good one. |
16:17 |
jeff |
senator: it appears that onmethoderror will fire, but only in the staff client. also, "onerror" is not (always?) a less-specific handler for method errors. |
16:19 |
senator |
hrm. at least onmethoderror should work then for your cases of catching an event in the patron editor... odd |
16:19 |
jeff |
it does. my testing was incomplete. |
16:19 |
jeff |
i was updating my earlier assertion. |
16:20 |
jeff |
it works, but not in Firefox -- just in the staff client. i suspect I'm running into broken multipart as an issue, which is known and fixed. |
16:20 |
jeff |
so, probably-mostly mystery solved. |
16:21 |
dbwells |
berick: If you get a chance, I just posted an acq bug, bug #1239837. It is pretty simple, I think, but was causing us some grief today. |
16:21 |
pinesol_green |
Launchpad bug 1239837 in Evergreen 2.4 "Undefined variable prevents quick PO creation" (affected: 1, heat: 6) [High,New] https://launchpad.net/bugs/1239837 |
16:21 |
* dbwells |
heads out for a bit, will be back |
16:33 |
|
mrpeters left #evergreen |
16:34 |
jeff |
yeah, confirmed. it was multipart breaking. |
16:51 |
jeff |
and with multipart disabled in firefox, both onmethoderror and onerror fire. they also fire in the staff client as expected. |
16:52 |
jeff |
my test appears to have been sabatoged at least partly by muscle memory. when you are IN tab 3 in the client and you hold down alt and hit -, -, 3 -- there's very little clue that what actually happened was you did not clear cache, but you switched to tab 3. :-) |
16:58 |
|
akilsdonk joined #evergreen |
17:36 |
|
hopkinsju joined #evergreen |
17:37 |
hopkinsju |
Good afternoon gents. Can someone help me learn how to use the newish Org Unit Hiding feature in the OPAC? |
14:54 |
|
remingtron joined #evergreen |
14:54 |
eeevil |
dbwells: yes, a joiner of <empty-string> instead of NULL should do exactly what dbs wants in title|proper |
14:54 |
dbwells |
eeevil: IIRC, using the joiner of '' would fail when the call-template "part" made nodes (that is, they would stick together too). I thought about stringifying that too, but called it a day instead. |
14:54 |
* eeevil |
looks |
14:55 |
eeevil |
arg ... yeah... you're right |
14:55 |
eeevil |
ok, separate for now ... more thinking on this for .NEXT |
14:56 |
eeevil |
dbwells: ok. I've exhausted all complaints with the branch. there's too much to do to make it perfect, and it's good enough. :) |
14:56 |
eeevil |
dbwells++ |
14:56 |
eeevil |
and |
14:56 |
eeevil |
bshum++ |
14:56 |
eeevil |
senator++ |
14:56 |
eeevil |
for testing |
14:57 |
dbwells |
eeevil: we arrived at exactly the same place :) |
14:59 |
dbwells |
eeevil: I do want to continue down the better road for .NEXT, and really, the sooner, the better. 3-4 months out I am not likely to remember any of this. |
14:59 |
berick |
phasefx: finally, is this expected when running the tests? "Subroutine section_pkg redefined at (eval 1493) line 4."? -- seems to repeat w/ each test. |
14:59 |
berick |
they pass OK |
15:01 |
dbwells |
eeevil: I am also very keen to start documenting all the reasons for what we are doing, and I tried to do some of that on the bug / in the commits. It's definitely the sort of case where you try to push down one thing and something pops out on the other side, so to speak. |
15:01 |
phasefx |
berick: it happens with however make livecheck does it, but does not happen if you run the tests directly out of the live_t/ directory with perl or prove. I didn't try very hard to figure out why |
15:01 |
|
rfrasur joined #evergreen |
15:02 |
* phasefx |
blindly based livecheck off of what check was doing, but it may not make sense to do it that way |
15:02 |
berick |
phasefx: copy that. i'll leave it for another day -- maybe the same day we fix uninitialized value $dup_args{"real_api_name"} |
15:02 |
berick |
(from make check) |
15:03 |
phasefx |
whenever someone goes on a holy mission to eliminate all warnings :) |
15:05 |
berick |
skabam! live tests. |
15:05 |
* berick |
tries to think of a good live test to add |
15:07 |
phasefx |
berick++ |
15:07 |
pinesol_green |
[evergreen|Jason Etheridge] tests against stock test data and live Evergreen - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=310a0d5> |
15:07 |
pinesol_green |
[evergreen|Jason Etheridge] test bill payment - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=5c883f0> |
15:07 |
pinesol_green |
[evergreen|Jason Etheridge] Add a TestUtils library for live tests - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=6530fb4> |
15:08 |
phasefx |
berick: we could use examples for how to use MARC in a perl test |
15:10 |
berick |
phasefx: as in, inspect bre.marc (e.g. via xml::Libxml) or specifically marc::record? |
15:11 |
phasefx |
berick: let's say someone wants to test some parameter to an opensrf marc update method, how might they represent/get the MARC they're using in the perl environment? |
15:14 |
berick |
phasefx: k, i can probably come up w/ something. gracias |
15:14 |
eeevil |
dbwells: indeed. we've turned search into a stress ball! ;) |
15:15 |
dbwells |
heh |
16:05 |
* Dyrcona |
knows there are Princess Bride related jokes in all of these ouses. |
16:05 |
jboyer-isl |
It does exist and I've set and cleared it to no avail. I'm finding a patron with claimed items now. |
16:07 |
jeff |
jboyer-isl: on the patron whose counts you pasted earlier... can you confirm that they have a circ with a null checkin_time and with stop_fines of CLAIMSRETURNED, and that circ has an xact_finish that is non-null? |
16:07 |
berick |
phasefx: is this at all what you were thinking? http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/berick/bre-marc-live-test (top commit) |
16:07 |
|
kbutler joined #evergreen |
16:09 |
jboyer-isl |
jeff: Yes. For the pasted account xact_finish is set (no fines) and checkin_time is null |
16:10 |
jeff |
jboyer-isl: okay. and on your dev system you have a patron with a circ matching that same criteria, and it is shown where in production is it not? |
16:19 |
jboyer-isl |
jeff: I suppose that means that tomorrow I can file a bug on it in LP. At least I can stop chasing geese. |
16:23 |
jeff |
it appears that the storage method is more accurate but slower, and is only used by legacy scripts. the IDL view could just have its logic fixed rather easily. |
16:25 |
jboyer-isl |
That's good. I may see if I can get it working on our dev server tomorrow unless someone beats me to it. I'm out today in a couple mins. |
16:26 |
jeff |
it's an edit to fm_IDL.xml and some testing. :-) |
16:27 |
jboyer-isl |
Splendid. |
16:28 |
jboyer-isl |
thanks for talking it out, it would likely have taken me longer to find my mistake on my own. |
16:28 |
jboyer-isl |
jeff++ |
12:33 |
bshum |
phasefx_: It's not unprecedented, but if there anything in particular that doing so would gain us? |
12:33 |
bshum |
*is there |
12:33 |
bshum |
Just curious why we wouldn't just use whichever packaged version was available to us from indexdata |
12:34 |
phasefx_ |
bshum: I'm using berick's eg_wheezy_installer.sh script to install Evergreen from scratch periodically and run tests. It gets derailed whenever ftp.indexdata.dk is down, which is often enough to annoy me |
12:36 |
phasefx_ |
of course, it's working now :) |
12:36 |
bshum |
I guess I don't really have a strong opinion either way. |
12:37 |
bshum |
Though I would have wondered why the packaged yaz wasn't working for Wheezy |
12:37 |
phasefx_ |
may never have been tested |
12:37 |
bshum |
Though also, the packaged yaz for Precise has issues with it, so I guess I can't talk about that too much either. |
12:41 |
phasefx_ |
you know what, it's not yaz, it's simple server |
12:42 |
|
mrpeters joined #evergreen |
13:38 |
|
smyers_ joined #evergreen |
13:48 |
|
kbutler joined #evergreen |
14:01 |
Dyrcona |
cstore starts. cstore connects to the database. cstore dies silently. |
14:02 |
jeff |
try setting ON_ERROR_EXPLODE=1 |
14:03 |
jeff |
"push to test. *click* release to detonate." |
14:03 |
jeff |
Dyrcona: is this under some exceptional / test scenario, or in production? |
14:04 |
Dyrcona |
I'm just going to rebuild the VM with the old RAM parameters. |
14:04 |
Dyrcona |
jeff: It is a development VM with 4GB of RAM, and immediately after running osrf_control to start everything. |
14:04 |
Dyrcona |
reporter-store does the same thing, too. |
14:14 |
jeff |
good luck. i am interested from afar. |
14:15 |
Dyrcona |
I wonder if it is a memory limitation, but I don't see OOM killer messages. |
14:16 |
Dyrcona |
swap was not being used, and there were about 600MB of free RAM, out of 4GB. |
14:19 |
yboston |
Within 15 minutes I want to try an "on air" Google Hangout as preparation of the asciidoc training I want to give. Any volunteers to join me? |
14:19 |
yboston |
Your can send me your Google linked email accounts through private message. Thanks in advance |
14:34 |
yboston |
Any one see this test stream? http://youtu.be/fnH71b6rSWo |
14:34 |
jeff |
I'm not in an environment to participate or listen, but I can see the video, yes. |
14:35 |
kmlussier |
yboston: I see it! |
14:37 |
kmlussier |
yboston: I also see the presentation screen. But, like jeff, I'm not in a position to join the Hangout at the moment. |
15:37 |
Dyrcona |
the first object does, but the one being fleshed doesn't. |
15:38 |
Dyrcona |
should it work with a search? |
15:38 |
Dyrcona |
I'll try money::billing lacks pcrud. |
15:39 |
Dyrcona |
I'll test with the target copy which I know does.... |
15:41 |
Dyrcona |
So, if you search and request a flesh of something that doesn't have pcrud support, you get nothing. |
15:41 |
Dyrcona |
If you retrieve and flesh the same thing, you get the object you're retrieving, but the flesh_field is empty. |
15:45 |
* Dyrcona |
goes back to the original idea of using cstore. |
11:10 |
ElliotFriend |
awesome thought-process, but I'm 90% confident I would do more damage than good by modifying anything in Perl haha |
11:11 |
robertl_ |
Hi. Anyone knows how long it takes for beta release before it becomes stable? |
11:12 |
RoganH |
robertl_: as soon as bugs are squashed |
11:13 |
RoganH |
The beta blocker list is pretty small, then we get RC. How long something stays at RC depends partly on how much testing it gets. |
11:22 |
robertl_ |
thanks Rogan |
11:50 |
|
Dyrcona joined #evergreen |
11:52 |
Dyrcona |
pinesol_green: Any laters for me? |
12:46 |
pinesol_green |
[evergreen|Dan Wells] Stamping upgrade 0839: alternative title index fix - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=b8ec73d> |
12:47 |
jeff |
dbwells++ |
12:48 |
* Dyrcona |
is back |
12:52 |
jeff |
bah. pound appears to be unsuitable for use in environments where your local interface IP is not your public-facing IP, unless you go down the path of split dns. |
12:53 |
jeff |
perhaps exclusively where the local interface IP can change. |
12:53 |
jeff |
and another option is to update pound.cfg dynamically. |
12:59 |
jeff |
oh that's weird. re-test server behavior seems to skip/cache the version check. |
13:11 |
* tsbere |
hates library vendors that refuse to change how they send email because it worked the way they are when they started decades ago... |
13:16 |
Dyrcona |
@later tell gmcharlt Would you be interested in records that appear to OK in UTF-8 but cause warnings when converted to MARC-8 with MARC::Charset? |
13:16 |
pinesol_green |
Dyrcona: The operation succeeded. |
15:15 |
Dyrcona |
Hope I didn't step on backporting 0839, or is it for 2.5+ only? |
15:18 |
berick |
@later tell dbwells since we're in beta now, can we get an origin/rel_2_5 branch so master can remain open for business? |
15:18 |
pinesol_green |
berick: The operation succeeded. |
15:19 |
dbwells |
berick: I think in the past we have deliberately closed master until final release to encourage more testing and less new stuff. Maybe I am mistaken, though. |
15:19 |
dbwells |
berick: And by closed, I mean kept for bugfixes only. |
15:22 |
kmlussier |
It's been a while since I updated that page. I also added fparks_ and yboston to http://wiki.evergreen-ils.org/doku.php?id=contributing:contributors. fparks++ yboston++ |
15:22 |
kmlussier |
new_contributors++ |
15:22 |
berick |
hm, i think that's great as a policy -- let's focus on bug fixes - i'm not fond of enforcing it in that manner, though |
15:44 |
roses |
Dyrcona: I'm going to steal that line and tell them that!! |
15:44 |
Dyrcona |
Crazy staff.... |
15:44 |
bshum |
"I don't recognize this account, I'm gonna delete it." |
15:44 |
roses |
bshum: sounds like something i would do to myself while "testing" |
15:44 |
jeff |
All disclaimers about deleting users aside, to answer the original question, a staff user deleting another user will need DELETE_USER at the target user's home_ou, and may also need ACTOR_USER_DELETE_OPEN_XACTS.override at the target user's home_ou as well. You'll also need the same group application permissions that you would need to be able to edit the user in the first place -- for a stock install, that means you'd need group_application.user.patron fo |
15:45 |
jeff |
oh, and that truncated. if someone would kindly tell me where, i can re-send. |
15:45 |
bshum |
group_application.user.patron for... |
16:15 |
pinesol_green |
Launchpad bug 1097399 in Evergreen 2.4 "Reserve room for system config.metabib_field values" (affected: 4, heat: 24) [Medium,Confirmed] |
16:16 |
bshum |
I knew it existed somewhere |
16:17 |
dbwells |
bshum: very interesting, so it isn't in yet. Bummer. Same problems, different day :) |
16:18 |
bshum |
Dyrcona: Fwiw, my test system that I setup last Saturday only goes to id 30. With 30 being LCCN, so something is awry in there. |
16:19 |
bshum |
dbwells: Yeah, I think it was covered in an upgrade SQL during the 2.0 era. |
16:19 |
bshum |
dbwells: So folks who upgraded in that time were covered, but folks who went to Evergreen or always were on master were broken by the lack of proper sequencing. |
16:21 |
bshum |
dbwells: I'm curious about something (which I guess I would see if I did apply the branch), but... |
16:21 |
dbwells |
bshum: you must be right, because our custom indexes start at 101. |
16:21 |
bshum |
dbwells: What should the title data look differently between browse vs. regular? |
16:21 |
bshum |
*how should |
16:25 |
bshum |
Dyrcona: Yeah, the bug I linked to has a branch you created to do that ;) |
16:25 |
dbwells |
bshum: the bigger issue it fixes (which is why it is a blocker) is that the non-filing stuff was ending up in the sort_value in metabib.browse_entry. |
16:26 |
dbwells |
Dyrcona: so, you agree with yourself from 5 months ago, that's good :) http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commitdiff;h=95251643c7a60945f50b30be7e328ecbcae38cd6 |
16:26 |
Dyrcona |
I can still test it, though. |
16:28 |
Dyrcona |
That's amazing. These days I'm lucky if I agree with myself from 5 minutes ago. |
16:28 |
|
Bmagic joined #evergreen |
16:28 |
dbwells |
:) |
16:46 |
bshum |
dbwells: That helps, thanks! Sorry, I got distracted reminiscing about the old days. |
16:47 |
|
sseng__ joined #evergreen |
16:47 |
|
smyers_ joined #evergreen |
16:47 |
dbwells |
bshum: the second (more minor) change is that trailing punctuation for titles should be gone now from the title browse display (especially '/'). |
16:47 |
|
dconnor joined #evergreen |
16:48 |
|
smyers_ joined #evergreen |
16:51 |
|
dconnor joined #evergreen |
16:52 |
|
dconnor joined #evergreen |
16:53 |
dbwells |
bshum: Dyrcona: one thing I should also point out if you are watching the DB while testing is that metabib.browse_entry builds up dead rows. I don't think any code ever deletes them. I intend to open that as a separate bug, but it practice, I don't think it is a big deal. |
16:53 |
|
dMiller_ joined #evergreen |
16:56 |
mrpeters |
any syslog-ng experts left? i know most have switched to rsyslog but I am pulling my hair out trying to figure out why gateway and eg-stats logs are going to both their own log files, AND to osrfsys.$HOUR.log |
16:56 |
mrpeters |
its making troubleshooting a nightmare |
17:09 |
kmlussier |
Dyrcona: OK, well thanks for the attempt. :) |
17:09 |
* mrpeters |
now debates time investment vs switching them to rsyslog versus fixing syslog-ng |
17:13 |
Dyrcona |
kmlussier: Or maybe it will. A little regexp magic goes a long way. |
17:14 |
bshum |
dbwells: Interesting.... on the concerto test dataset from clean to running the reingest after applying the patch I do notice there are extra rows in metabib.browse_entry that don't appear in the metabib.browse_entry_def_map |
17:15 |
bshum |
I should have checked right before I ran it... doh |
17:16 |
dbwells |
bshum: right, those are the old rows which no longer apply. It will need a fix, but since those rows are likely to build up very slowly in real life, I don't want to block RC for it. |
17:16 |
bshum |
"slowly" |
17:16 |
* bshum |
goes to try his query in production for kicks :) |
17:28 |
dbwells |
bshum: I did this: |
17:28 |
dbwells |
select * from metabib.browse_entry mbe LEFT JOIN metabib.browse_entry_def_map mbedm ON mbedm.entry = mbe.id LEFT JOIN metabib.browse_entry_simple_heading_map mbeshm ON mbeshm.entry = mbe.id where mbedm.id IS NULL AND mbeshm IS NULL; |
17:29 |
dbwells |
I still found 5 things I wasn't expecting to be orphaned from the concerto set, so I am not sure what is going on there. |
17:30 |
bshum |
What is simple_heading_map? |
17:30 |
* bshum |
doesn't seem to have that in his production DB... |
17:31 |
bshum |
Oh I see |
17:31 |
bshum |
We don't have it yet, nevermind. |
17:31 |
bshum |
That went in after our last step forward |
17:37 |
bshum |
dbwells: Is there supposed to be a difference in how 6 vs. 31 metabib field present in the metabib.title_field_entry? Or are they basically identical and the question is just how they are rendered in the browse table values? |
17:38 |
bshum |
Having only just run that against the concerto test set quickly, the only 7 bibs that show differences in those indexed values were like: |
17:38 |
bshum |
L'unique et sa propriété vs. L' unique et sa propriété |
17:38 |
bshum |
The extra space gets removed, and so forth |
17:39 |
bshum |
Otherwise, every other 6 and 31 were exactly the same on every bib in the test dataset. |
17:39 |
* bshum |
needs more test databases |
17:40 |
dbwells |
bshum: I am totally confused, as that is a change I would not expect to see. |
17:40 |
bshum |
dbwells: Well, I didn't inspect very closely, I just ran the final upgrade SQL from the branch on the bug ticket. |
17:40 |
bshum |
Maybe I missed something |
17:41 |
pinesol_green |
Launchpad bug 1235497 in Evergreen "Title browse has display and sorting issues" (affected: 1, heat: 6) [Undecided,New] |
17:41 |
dbwells |
bshum: maybe you still have the old 31? |
17:41 |
dbwells |
(though I assume the upgrade would have blown up at you) |
17:41 |
bshum |
dbwells: The test system I applied the upgrade to didn't have a 31 in it before I applied the sql. |
17:42 |
senator |
"the baroque concerto" moves from the wrong place to the right place in my test |
17:42 |
bshum |
It's a fresh DB as of last Saturday, but I can rebuild again just in case. |
17:42 |
senator |
and similar |
17:42 |
bshum |
Oops, maybe tomorrow. (just saw the clock) |
17:45 |
dbwells |
senator: thanks for the testing and signoff. Glad it worked for you. |
17:48 |
|
RBecker joined #evergreen |
17:51 |
dbwells |
senator: I also responded to Liam before I saw your comment. We said roughly the same thing, save one point. The stock MODS <titleInfo> field does have the <nonSort> tag, so the main hang up is that we only get one 'joiner' for everything in title, while we (ideally) need a different joiner from <nonSort>-><title> than from <title>-><subtitle>. |
17:52 |
dbwells |
maybe I misunderstood what you were saying. In any case, it is a topic for another day. |
17:53 |
* dbwells |
hopes not tomorrow |
17:56 |
senator |
heh |
17:56 |
|
smyers_ joined #evergreen |
17:57 |
senator |
ah k, well, that is a significant point |
12:31 |
dbwells |
kmlussier: yes, I plan to redo them again. The way I did it for the beta was pretty ad hoc, so hopefully the second time through I can be a little more systematic about it. |
12:32 |
kmlussier |
dbwells: Great! That's what I was hoping you would say. :) |
12:35 |
|
yboston joined #evergreen |
12:38 |
bshum |
dbwells: So we're testing that fix for receiving |
12:38 |
bshum |
dbwells: And it looks like it received properly and we created the item |
12:38 |
bshum |
But apparently the range in the summary holdings doesn't show the updated issue |
12:40 |
dbwells |
bshum: what holdings display are you using? |
12:40 |
bshum |
Mary was looking at http://acorn.biblio.org/eg/opac/record/2236299?expand=issues;sid=1644;stype=basic;sepath=2013;sepath=09;seoffset=0;seoffset=0;seoffset=0#issues for example. |
12:40 |
bshum |
Where the last issue that was just added was the Sept 15 one |
12:46 |
dbwells |
I should probably just say "Oh, 'use fully compressed holdings'? That's senator's problem." ;) |
12:50 |
|
mllewellyn joined #evergreen |
12:58 |
_bott_ |
I hate weird problems. |
12:58 |
dbwells |
bshum: I can see what you are saying on the record you posted, but I can't duplicate it on my test box. I'd need to see the exact holding codes for your Sep.08 and Sep.15 issues, I guess, then go from there. |
12:59 |
* dbwells |
goes to lunch, back in a bit |
12:59 |
_bott_ |
Has anyone found the status_changed_time trigger not firing consistently? I have random cases where the status is changed at checkin, but the status_changed_time is not updated, it's left at the checkout time. This in turn screws up reshelving delay. |
13:08 |
bshum |
dbwells++ # we figured it out, it was a local holding_code problem |
13:08 |
bshum |
The holding_code had been modified with a bad copy/paste and the date in the code was 08 not 15 |
13:08 |
bshum |
User error |
13:08 |
bshum |
We'll test again with another sample, but I think this means your patch works to resolve the receiving problem. |
13:10 |
|
ericar joined #evergreen |
13:11 |
pinesol_green |
[evergreen|Kathy Lussier] Move various release notes into correct directory - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=30a69c4> |
13:11 |
pinesol_green |
[evergreen|Kathy Lussier] Adding new OPAC features to Release Notes. - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=e66a212> |
13:20 |
mllewellyn |
pleading guilty to the operator causing the error. |
13:31 |
|
RoganH joined #evergreen |
13:42 |
dbwells |
bshum: yay :) |
13:42 |
bshum |
dbwells: Yep, she tested on a different issue and it received perfectly fine. |
13:42 |
bshum |
I'll sign off on your fix and get that pushed to master. |
13:43 |
bshum |
dbwells++ |
13:45 |
rfrasur |
jcamins: this speaker is talking exactly what you were talking about earlier... |
13:46 |
pinesol_green |
[evergreen|Dan Wells] Relax MFHD subfield 'a' requirement for caption/patterns - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=a420b0e> |
13:46 |
jcamins |
rfrasur: oh? What speaker? |
08:38 |
Dyrcona |
I use IndexData's repo for debs by way of explanation. |
08:39 |
|
Shae joined #evergreen |
08:41 |
bshum |
Dyrcona: libyaz5 as a client seemed fine so far in my light testing. |
08:42 |
Dyrcona |
bshum: I want to test it with the server. |
08:42 |
bshum |
I haven't gotten to try it with the server part yet. |
08:42 |
bshum |
Cool. |
08:43 |
bshum |
Also, super exciting for Marque test. |
08:44 |
Dyrcona |
Yeah. I figure if you want to use Marque.pm for Boopsie, then go ahead. |
08:44 |
Dyrcona |
It all has to be installed by hand, but the final version in the branch on lp1223903 will do all the right things. |
08:45 |
Dyrcona |
bug 1223903 |
11:47 |
|
jdouma joined #evergreen |
11:55 |
|
artunit joined #evergreen |
11:57 |
|
smyers_ joined #evergreen |
12:03 |
Dyrcona |
bshum++ # for testing before I get to ti. |
12:03 |
Dyrcona |
it |
12:07 |
Dyrcona |
bshum: We might be able to trap the MARC::File error with an eval and print the bib id. |
12:08 |
* Dyrcona |
was away from his desk for the past hour or more, chatting with a coworker about Evergreen issues and our new member library's first day. |
14:13 |
depesz |
http://depesz.privatepaste.com/82ebcbeadf - that's the original |
14:14 |
depesz |
http://depesz.privatepaste.com/350bbaf122 -> that's the new one |
14:14 |
eeevil |
depesz: I'm looking at master now ... at a real computer today :) |
14:14 |
depesz |
YAY :) |
14:15 |
depesz |
eeevil: anyway for some particular set of data, and query - original functions does it's work in ~ 400ms, and new one in ~ 16ms |
14:16 |
depesz |
of course 400ms is not that bad, but - when I tested pg logs for all slow queries, the one that used the most time was call to "unapi.bre()" function - not sure if this is part of evergreen. |
14:16 |
eeevil |
it is |
14:16 |
depesz |
when I profiled it - 98% of its runtime was call to unapi.holdings_xml, which in turn had bulf of its time from evergreen.ranked_volumes |
14:17 |
depesz |
so - while the immediate speedup is perhaps not all that great, it looks like it should optimize very often (in this system) called query. |
16:25 |
senator |
my verbiage, if you can parse it, should incidentally explain why you never really saw a difference with Callender's branch |
16:26 |
senator |
although it should have worked too |
16:26 |
|
Dyrcona left #evergreen |
16:26 |
bshum |
senator: Yeah I saw the changes for the actual index in yours. I'm not entirely sure what I was testing back then and all. It seems like forever ago now. |
16:26 |
senator |
no, that's separate |
16:26 |
bshum |
senator: That and bibcn is like the worst search in a consortium that doesn't share any call numbers uniformly :D |
16:27 |
bshum |
So it hasn't been a huge priority on my end of things. |