duongkame commented on PR #1007: URL: https://github.com/apache/ratis/pull/1007#issuecomment-1886366066
> > ... a StateMachine may cache LogEntry's data and use it outside the Ratis control. ... > > We should pass the `ReferenceCountedObject` in this case. Indeed, Ozone currently cache log entries in datanode. We should make it work. Yeah, it's where I'm from. Agree, I filed RATIS-1997 for this. Btw, for backward compatibility, we need to disable zero copy by default. When applications enable zero-copy, a migration to use the new versions of StateMachine API is needed. -- 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]
