[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: RE : RE : [sdx-developers] Changements récents
From: |
Pierrick Brihaye |
Subject: |
Re: RE : RE : [sdx-developers] Changements récents |
Date: |
Wed, 19 May 2004 09:18:29 +0200 |
User-agent: |
Mozilla/5.0 (Windows; U; Win98; fr-FR; rv:1.6) Gecko/20040113 |
Salut,
Rasik Pandey a écrit :
Je ne suis pas contre faire rentrer ça dans fr.gouv.culture.sdx
mais...
comment gérer le versioning du QueryParser ?
Pour l'instant je ne vois pas d'autres méthodes qu'à la main.
Moi non plus :-)
Justement je voulais garder DefaultQueryParser parce que je pensais que c'était
plus facile à synchroniser (à la main) avec le QueryParser.jj de Lucene que
ZeroPositionIncrementQueryParser. Mais ça se discute....peut-être on pourrait
mieux factoriser ton code dans ZeroPositionIncrementQueryParser (hors la
méthode getFieldQuery) pour le rendre plus facile à synchroniser à la main,
puis on vire DefaultQueryParser???
Aucun problème !
Pour l'instant on déplace UnanalyzedQuery vers
fr.gouv.culture.sdx.lucene.query.UnanalyzedQuery et donc on change les
QueryParser(s) pour utiliser cette classe.
Moi je veux bien mais avant de faire le refactoring, j'aurais souhaité
que les archives Lucene aient :
1) un patch avec une UnanalyzedQuery. Je suis à peu près certain qu'il y
a un besoin réel dans ce domaine et que la syntaxe employée est
acceptable (vérifier tout de même que l'on peut escaper le "|"). Si ça
embraye... je veux bien patcher la doc et les tests :-)
2) un patch avec le ZeropositionIncrementQueryParser (dans la veine de
http://issues.apache.org/bugzilla/show_bug.cgi?id=23307). Ici encore, on
pourrait également patcher les tests.
Puis on n'aura plus besoin de patché le jar de Lucene dans le build.xml de SDX.
Je comprends bien est c'est effectivement l'objectif. En fait je n'ai
jamais réellement compris pourquoi j'ai eu besoin de patcher.
Normalement, si le même ClassLoader est utilisé, on devrait avoir accès
à org.apache.lucene.query.UnanalyzedQuery même si cette classe est dans
sdx.jar.
Soit je me trompe, soit il y a plusieurs ClassLoaders, soit j'ai levé un
bug dans la JVM sur laquelle j'ai testé l'analyseur arabe.
A+
--
Pierrick Brihaye, informaticien
Service régional de l'Inventaire
DRAC Bretagne
mailto:address@hidden
+33 (0)2 99 29 67 78