[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-23 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-15 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-14 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-02-22 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-02-18 Thread GitBox
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

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-02-16 Thread GitBox
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