update ChangeLog for OpenSRF 3.0.3
[OpenSRF.git] / ChangeLog
1 commit 5002dd8de58a7f3e79a14bf2ca28cb4d2b7ae71a
2 Author: Galen Charlton <gmc@equinoxinitiative.org>
3 Date:   Fri Jun 7 17:10:11 2019 -0400
4
5     update versions for OpenSRF 3.0.3
6     
7     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
8
9 7       7       README
10 2       2       src/perl/lib/OpenSRF.pm
11 1       1       src/python/setup.py
12 1       1       version.m4
13
14 commit 9124bf3c82ad29e20a0a2607cb78d6b9db325efa
15 Author: Galen Charlton <gmc@equinoxinitiative.org>
16 Date:   Fri Jun 7 17:08:30 2019 -0400
17
18     update release notes for OpenSRF 3.0.3
19     
20     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
21
22 29      0       doc/RELEASE_NOTES.txt
23
24 commit ca876a9ae69ed49369d8722831f11c3876d893ad
25 Author: John Merriam <jmerriam@biblio.org>
26 Date:   Wed Apr 10 11:06:14 2019 -0400
27
28     LP#1824184: Change potentially slow log statements to subroutines
29     
30     The OpenSRF code was searched for potentially slow logging statements at
31     log level info or above. We then changed those logging statements to be
32     delayed execution subroutines. This is in reaction to LP1823338 where a
33     slow debug logging statement was slowing down SIP checkins even though
34     debug logging was not turned on.
35     
36     Here is some pseudocode that shows what is being done here:
37     
38     $log->debug("Some text " . $some->slow_method);
39     
40     would be changed to:
41     
42     $log->debug(sub{return "Some text " . $some->slow_method });
43     
44     With this change, an unnamed sub is passed to the OpenSRF logger module
45     and it will not be executed unless the global logging level is set to
46     debug or higher (the slow_method will not be called unless it is needed
47     for that logging level).
48     
49     ********
50     If/when this is committed, please use delayed execution subroutines for
51     any logging statements that could be slow in the future. It is recommend
52     that any logging statements that do not consist entirely of quoted text
53     and/or already available scalar variables use delayed execution
54     subroutines.
55     ********
56     
57     Testing notes
58     -------------
59     [1] This patch reduced the duration of open-ils.storage.actor.user.total_owed
60         calls on a test system from an average of 0.13 seconds to an overage
61         of 0.009 seconds.
62     [2] It didn't materially affect the duration of calls to
63         open-ils.actor.user.opac.vital_stats.
64     [3] This make sense: open-ils.storage has far more generated methods
65         than open-ils.actor.
66     [4] There are enough instances in Evergreen of using method_lookup to find
67         routines in open-ils.storage that the speed improvement may well be
68         perceptible to humans, not just SIP sorters.
69     
70     Signed-off-by: John Merriam <jmerriam@biblio.org>
71     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
72
73 7       7       src/perl/lib/OpenSRF/AppSession.pm
74 9       9       src/perl/lib/OpenSRF/Application.pm
75 4       4       src/perl/lib/OpenSRF/DomainObject/oilsMessage.pm
76 2       2       src/perl/lib/OpenSRF/EX.pm
77 3       3       src/perl/lib/OpenSRF/MultiSession.pm
78 9       9       src/perl/lib/OpenSRF/Server.pm
79 1       1       src/perl/lib/OpenSRF/Transport.pm
80 1       1       src/perl/lib/OpenSRF/Transport/SlimJabber/Client.pm
81
82 commit 07ae6134166a804a5ba86c379ef5dec00577a873
83 Author: Galen Charlton <gmc@equinoxinitiative.org>
84 Date:   Fri Jun 7 11:30:42 2019 -0400
85
86     LP#1824181: add test cases
87     
88     To test
89     -------
90     [1] Verify that 'make check' (or prove src/perl/t/09-Utils-Logger.t)
91         passes.
92     
93     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
94
95 10      1       src/perl/t/09-Utils-Logger.t
96
97 commit 327a272e1d5fa187f1c78afc963d125e3bb32860
98 Author: John Merriam <jmerriam@biblio.org>
99 Date:   Wed Apr 10 09:51:54 2019 -0400
100
101     LP#1824181: Allow 1st arg to logger to be string or subroutine
102     
103     This simple change allows the $msg passed to _log_message in Logger.pm
104     to be either a regular string or a delayed exec subroutine. This is in
105     reaction to LP1823338 where a slow debug logging statement was slowing
106     down SIP checkins even though debug logging was not turned on.
107     
108     With this change you can take this:
109     
110     $log->debug("Available methods\n\t".join("\n\t", keys %{ $_METHODS[$proto] }), INTERNAL);
111     
112     and change it to this:
113     
114     $log->debug(sub{return "Available methods\n\t".join("\n\t", keys %{ $_METHODS[$proto] }) }, INTERNAL);
115     
116     and then that slow debug logging line will not be executed unless
117     debug logging is turned on.
118     
119     Signed-off-by: John Merriam <jmerriam@biblio.org>
120     Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
121
122 3       0       src/perl/lib/OpenSRF/Utils/Logger.pm