[Koha-bugs] [Bug 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Julian Maurice  changed:

   What|Removed |Added

 Resolution|--- |WONTFIX
 Status|In Discussion   |RESOLVED

--- Comment #13 from Julian Maurice  ---
(In reply to Julian Maurice from comment #12)
> Since caching the XSLT result is not desirable, should we close this bug ?

No answers in 3 months, so I'm changing status to WONTFIX. Feel free to reopen
if you are willing to work on this.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #12 from Julian Maurice  ---
Since caching the XSLT result is not desirable, should we close this bug ?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Katrin Fischer  changed:

   What|Removed |Added

 CC||katrin.fisc...@bsz-bw.de

--- Comment #11 from Katrin Fischer  ---
As far as I know there has been no official decision on Zebra. There are still
Zebra bugs being worked on.

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #10 from David Cook  ---
(In reply to Tomás Cohen Arazi from comment #9)
> Zebra support is in freeze mode. 

That's good to hear. I've been wondering how long we'd try to support both. I
look forward to a time where we're just focusing on one.

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #9 from Tomás Cohen Arazi  ---
(In reply to David Cook from comment #8)
> (In reply to Tomás Cohen Arazi from comment #7)
> > (In reply to Fridolin SOMERS from comment #5)
> > > This will also avoid indexing all record on item circulation.
> > 
> > Not sure about it, but we could certainly index items and make JOIN
> > operations in our queries ;-) (for limits, as David also suggests).
> 
> I was totally thinking about that, as I was thinking about Solr query joins.
> I imagine ElasticSearch must be able to do the same thing, although I don't
> think Zebra could. Or did you have something in mind for JOIN operations?

I was thinking ES. Zebra support is in freeze mode. And I'm sure you cannot mix
different databases on a query.

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #8 from David Cook  ---
(In reply to Tomás Cohen Arazi from comment #7)
> (In reply to Fridolin SOMERS from comment #5)
> > This will also avoid indexing all record on item circulation.
> 
> Not sure about it, but we could certainly index items and make JOIN
> operations in our queries ;-) (for limits, as David also suggests).

I was totally thinking about that, as I was thinking about Solr query joins. I
imagine ElasticSearch must be able to do the same thing, although I don't think
Zebra could. Or did you have something in mind for JOIN operations?

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #7 from Tomás Cohen Arazi  ---
(In reply to Fridolin SOMERS from comment #5)
> (In reply to Tomás Cohen Arazi from comment #4)
> > I have a concern: if we cache the record to be displayed, then the item
> > statuses won't be up-to-date. If the tradeoff is not enough, maybe this is
> > just not worth. Because people expect to see updated information on the
> > records.
> 
> Indeed.
> 
> I think we need to change this by moving items out of this XSLT file.
> I think we should use Ajax to load in realtime items infos, with and API for
> that.

I agree! Let's have that endpoint!

> This will also avoid indexing all record on item circulation.

Not sure about it, but we could certainly index items and make JOIN operations
in our queries ;-) (for limits, as David also suggests).

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #6 from David Cook  ---
(In reply to Fridolin SOMERS from comment #5)
> (In reply to Tomás Cohen Arazi from comment #4)
> > I have a concern: if we cache the record to be displayed, then the item
> > statuses won't be up-to-date. If the tradeoff is not enough, maybe this is
> > just not worth. Because people expect to see updated information on the
> > records.
> 
> Indeed.
> 
> I think we need to change this by moving items out of this XSLT file.
> I think we should use Ajax to load in realtime items infos, with and API for
> that.
> This will also avoid indexing all record on item circulation.
> Dreams ...

I often think using Ajax to load real time item info is a good idea too,
although I think we'd still have the indexing issue because of the "limit to
currently available items" filter on the search. 

Often, I think we should think about our OPAC as if it were a totally separate
system, and just link it up to the Koha instance using APIs.

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #5 from Fridolin SOMERS  ---
(In reply to Tomás Cohen Arazi from comment #4)
> I have a concern: if we cache the record to be displayed, then the item
> statuses won't be up-to-date. If the tradeoff is not enough, maybe this is
> just not worth. Because people expect to see updated information on the
> records.

Indeed.

I think we need to change this by moving items out of this XSLT file.
I think we should use Ajax to load in realtime items infos, with and API for
that.
This will also avoid indexing all record on item circulation.
Dreams ...

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Fridolin SOMERS  changed:

   What|Removed |Added

 CC||fridolin.som...@biblibre.co
   ||m

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Julian Maurice  changed:

   What|Removed |Added

   Assignee|julian.f...@biblibre.com|julian.maur...@biblibre.com
 CC||julian.maur...@biblibre.com

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

David Cook  changed:

   What|Removed |Added

 CC||dc...@prosentient.com.au

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #3 from Gaetan Boisson  ---
Works for me, although the performance gain was not as big as expected. I will
test it again on a larger data set.

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Gaetan Boisson  changed:

   What|Removed |Added

 CC||gaetan.bois...@biblibre.com
  Attachment #40740|0   |1
is obsolete||

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Tomás Cohen Arazi  changed:

   What|Removed |Added

 Status|Signed Off  |In Discussion
 CC||tomasco...@gmail.com

--- Comment #4 from Tomás Cohen Arazi  ---
I have a concern: if we cache the record to be displayed, then the item
statuses won't be up-to-date. If the tradeoff is not enough, maybe this is just
not worth. Because people expect to see updated information on the records.

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #2 from Gaetan Boisson  ---
Created attachment 42179
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42179=edit
[SIGNED-OFF] bug 14476 : Caching C4::XSLT::transformMARCXML4XSLT

This patch is improving the opac-search performances by
caching the results of C4::XSLT::transformMARCXML4XSLT

My tests (with Devel::NYTProf) showed a gain of about
65% on C4::XSLT::transformMARCXML4XSLT and
35% on C4::XSLT::XSLTParse4Display.

The gain of performances is increasing with the numbers
of results.

Signed-off-by: Gaetan Boisson 

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Gaetan Boisson  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

Julian FIOL julian.f...@biblibre.com changed:

   What|Removed |Added

 Status|ASSIGNED|Needs Signoff

-- 
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 14476] Improving opac-search performances

[bugzilla-daemon]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476

--- Comment #1 from Julian FIOL julian.f...@biblibre.com ---
Created attachment 40740
  --
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40740action=edit
bug 14476 : Caching C4::XSLT::transformMARCXML4XSLT

This patch is improving the opac-search performances by
caching the results of C4::XSLT::transformMARCXML4XSLT

My tests (with Devel::NYTProf) showed a gain of about
65% on C4::XSLT::transformMARCXML4XSLT and
35% on C4::XSLT::XSLTParse4Display.

The gain of performances is increasing with the numbers
of results.

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