mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1076658125
@romseygeek There is something else in roadmap or future ideas regarding
this package that I can contribute? Or where I can look to contribute more to
the project. I'd like to rema
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1074048328
Ok!
--
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.
To unsubsc
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1073983473
Done!
Thanks for Your patience and support @romseygeek !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1073666978
@romseygeek sorry to bother you, just a reminder :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1067734639
@romseygeek I've merged main and restored listeners to the abstarct class,
You were right the sleep was very ugly
--
This is an automated message from the Apache Git Service.
To r
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1066549257
@romseygeek fixed !
--
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 comm
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1047984458
@romseygeek I should have fixed everything, also added few lines of docs to
explain read-only behaviour.
--
This is an automated message from the Apache Git Service.
To respond to
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1044239703
OK I've moved things around, making QueryIndex an abstract class sharing
common code for both implementations
ReadonlyQueryIndex have the same logic of in-memory laoding of parsed
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1041408992
@romseygeek I've updated with the requested changes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL