benwtrent commented on issue #13127: URL: https://github.com/apache/lucene/issues/13127#issuecomment-2264278817
Yeah, looking at `markForFullFlush`, it seems like we mark the generation to gather that `seqNo`, then unlock DWFC, and this allows new DWPT to be returned with the old generation being referenced, thus making the set `maxSeqNo` wrong. I guess the solution is for `getAndLock()` to not return a `dwpt` that references a flushed deleteQueue? @jpountz ^ I know this is a ton of information, but I am not sure what is correct behavior and since you were the last to touch this maybe you have better context -- 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 unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
