szetszwo commented on PR #924: URL: https://github.com/apache/ratis/pull/924#issuecomment-1728106815
> I'm wondering if an asynchronous thread would be a more elegant design. It looks like it would require a big change in cache concurrency control. The `daemon` in `AwaitToRun` is an asynchronous thread. > If my subsequent review holds true, perhaps we could contemplate managing it within the primary appendLog flow but trigger it less frequently, maybe when transitioning the raftlog. This could potentially reduce implementation complexity. > > What's your opinion? In `appendEntryImpl`, how about we evict cache only after `rollOpenSegment`? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
