Need feedback on possible patch to improve memory performance in Zend_Search_Lucene

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Need feedback on possible patch to improve memory performance in Zend_Search_Lucene

Adam Lundrigan
Hey all,

Could someone knowledgeable in the area of Zend_Search_Lucene take a look at the proposed fix for this performance issue with Zend_Search_Lucene_Analyzer_Token ?

http://framework.zend.com/issues/browse/ZF-3683

The gist of the change is that the provided token object would be updated in-place instead of having a new instance created and returned.  This area of Zend_Search_Lucene is not unit tested, so I don't know if this fix will have any negative repercussions further down the line.  If the fix is sound, I would like to get it into 1.12, as Zend_Search_Lucene is quite memory-intensive so any performance gain in that department is heartily welcomed.

Thanks,

--
Adam Lundrigan, B.Sc, ZCE
[hidden email]