[Koha-bugs] [Bug 7188] case-insensitive grep of /mysql/ in C4::Context
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7188 Jared Camins-Esakov jcam...@cpbibliography.com changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable --- Comment #4 from Jared Camins-Esakov jcam...@cpbibliography.com --- Included in 3.6 prior to 3.6.5. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 7188] case-insensitive grep of /mysql/ in C4::Context
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7188 --- Comment #1 from Ian Walls ian.wa...@bywatersolutions.com 2011-11-06 11:22:59 UTC --- Created attachment 6244 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=6244 Bug 7188: C4::Context::db_scheme2dbi does case-insensitive grep Checking NYTProf in C4::Context, it appears that the /i flag is highly time-consumming As we don't support anything outside from mysql, returning directly mysql directly If in a future version PostgresSQL works, then we will have to update this sub. But if an ORM is introduced, this code will be removed completly anyway Testing note: Drops execution time of the line from 300ms to 20 microseconds (in my testing). -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 7188] case-insensitive grep of /mysql/ in C4::Context
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7188 Ian Walls ian.wa...@bywatersolutions.com changed: What|Removed |Added Priority|P5 - low|PATCH-Sent Patch Status|--- |Passed QA --- Comment #2 from Ian Walls ian.wa...@bywatersolutions.com 2011-11-06 11:41:06 UTC --- This change reduces my installation from taking over 300ms to run this line, to 27 micro seconds. In my (limited) tests, I'm also finding that using /mysql/ returns in about the same amount of time as forcing the return of mysql, within a few micro seconds. I don't know why, but figured I'd check. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 7188] case-insensitive grep of /mysql/ in C4::Context
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7188 Paul Poulain paul.poul...@biblibre.com changed: What|Removed |Added Version|rel_3_8 |rel_3_6 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/