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

Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2021/05/25 11:40:13 UTC

[GitHub] [lucene] romseygeek commented on pull request #150: LUCENE-9204: Remove Spans references from DisiWrapper

romseygeek commented on pull request #150:
URL: https://github.com/apache/lucene/pull/150#issuecomment-847796594


   I had hoped to do something similar to what we did for ConjunctionUtils and pull things into shared public factory classes, but that ended up with some very nasty generics code (things like `DisiWrapper<T extends DisiWrapper>`) so I think just cloning the classes here is unfortunately the correct tradeoff to make.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org