You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@lucene.apache.org by Varun Thacker <va...@vthacker.in> on 2018/08/03 21:22:22 UTC

Remove SuppressCodecs Lucene3x Lucene4x?

Quite a few tests have this annotation :

@LuceneTestCase.SuppressCodecs({"Lucene3x",
"Lucene40","Lucene41","Lucene42","Lucene45"})

I don't think the following codecs exist anymore in the codebase and we
should just remove the annotation from tests that try suppressing these
codecs ?

   - Direct
   - Lucene3x
   - Lucene40
   - Lucene41
   - Lucene42
   - Lucene45
   - Appending

Re: Remove SuppressCodecs Lucene3x Lucene4x?

Posted by Adrien Grand <jp...@gmail.com>.

+1 to remove all annotations about legacy codecs

Le ven. 3 août 2018 à 23:22, Varun Thacker <va...@vthacker.in> a écrit :

> Quite a few tests have this annotation :
>
> @LuceneTestCase.SuppressCodecs({"Lucene3x",
> "Lucene40","Lucene41","Lucene42","Lucene45"})
>
> I don't think the following codecs exist anymore in the codebase and we
> should just remove the annotation from tests that try suppressing these
> codecs ?
>
>    - Direct
>    - Lucene3x
>    - Lucene40
>    - Lucene41
>    - Lucene42
>    - Lucene45
>    - Appending
>
>