The correct fix is to check the returned/calculated ID in the IndexCleaner, otherwise its racy.
Tue, Jul 16
Let's not argue but investigate and fix why this seemingly innocent patch caused the behavior described.
The bug report was online for two weeks and it was categorized as Baloo bug. You had more than enough time to notice it.
Why don't you notify the relevant people first
Sun, Jul 14
Sat, Jul 13
since this code is not compiled, I will merge it next week if noone objects
Thu, Jul 11
Apparently, it does fix bug 409257, which is pretty serious one (db corruption, after all).
Tue, Jul 9
I'd like to revisit this. Can we get back to it and fix the outstanding issues?
Sun, Jul 7
- remove extra space
Fri, Jul 5
Thu, Jul 4
No maintainer here, also no opinion :) Thus resigning to not keep wrong hopes for feedback from me.
friendly ping, I would like to get this into 5.60
Mon, Jul 1
Sun, Jun 30
- bring back openly coded counting
Note that it somewhat duplicates work done inside QueryParser. Which is almost not used anywhere - most of the parsing is done by AdvancedQueryParser.
Not entirely sure, but bug 409257 might be caused by that (at least in my case, it looked the same).
Sat, Jun 29
- adjust writer cmake
- use separate debug src