Re: nxlucene / pylucene2 / 64-bit

Jean-Marc Orliaguet wrote:
> Julien Anguenot wrote:
>> Hi Jean-Marc,
>>
>>   
> 
> Hi Julien,
>> Of course, using RedHat or Fedora is always the smartest choice you can
>> make regarding PyLucene... You won't get any troubles. Things are lot
>> more tricky on Debian based systems...
>>
>>  
>>> BTW, PyLucene-1.9.1 doesn't seem to work on 64-bit architectures
>>>     
>>
>> possible.
>>
>>  
>>> if anyone wants to test ...
>>>     
>>
>> Maybe it's time for a 1.x branch on PyLucene 1.9.x and put your branch
>> in the trunk ?
>>
>> I'm gonna check here if we can test out with some large data to validate
>>  your changes.
>>
>> Cheers,
>>
>>   
> 
> I'm going to test with real data the coming weeks. Eventually it would
> indeed be a good idea to upgrade nxlucene to support pylucene-2.0.0 by
> default.
> there is a  "- fixed memory leak with PyLucene.PythonThread instances"
> in the CHANGELOG between 1.9 and 2.0 for instance.

Yes, we can remove the patch I added related to this bug within NXLucene :

http://svn.nuxeo.org/trac/pub/browser/NXLucene/trunk/src/nxlucene/server/PatchPythonThread.py

> apart from very minor API changes:
> 
> - I replaced 'sys.maxint' with int(2**31-1) since on 64-bit systems you
> end up with 9223372036854775807 which java lucene chokes on.

ok :)

>  maybe there is a constant that can be imported from Java. in Jython
> there is  java.lang.Integer.MAX_VALUE which seems to be the value you
> want here.

Maybe. Has to be check.

> - a loop in nxlucene/server/core.py breaks on IndexError instead of
> PyLucene.JavaError:
> 
>    except PyLucene.JavaError:
>       break
> +  except IndexError:
> +     break

ok.

>  it seems to be related to:
>  "- changed Hits iteration to use Hit object yet work as before (for i,
> doc in..)"
> 
>  so I guess a standard 'for' loop can be used here instead.

ok.

	J.

-- 
Julien Anguenot | Nuxeo R&D (Paris, France)
Open Source ECM - http://www.nuxeo.com
Nuxeo 5 : http://www.nuxeo.org
Mobile: +33 (0) 6 72 57 57 66

_______________________________________________
cps-devel mailing list
http://lists.nuxeo.com/mailman/listinfo/cps-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGHRaRGhoG8MxZ/pIRAnAbAJ9m+PFJmtffmxp27ZsIRg1TTIqDBACggtFs
yQTZPp2sJweyYG+LZ5MfFkI=
=SVoo
-----END PGP SIGNATURE-----