szetszwo merged PR #1106:
URL: https://github.com/apache/ratis/pull/1106
--
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...@ratis.apache
szetszwo commented on PR #1106:
URL: https://github.com/apache/ratis/pull/1106#issuecomment-2144062303
@duongkame , "This pull request is still a work in progress". I guess it is
not the case?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
szetszwo commented on code in PR #1106:
URL: https://github.com/apache/ratis/pull/1106#discussion_r1622823140
##
ratis-common/src/main/java/org/apache/ratis/util/ReferenceCountedObject.java:
##
@@ -152,6 +153,10 @@ public boolean release() {
};
}
+ default ReferenceC
duongkame opened a new pull request, #1106:
URL: https://github.com/apache/ratis/pull/1106
## What changes were proposed in this pull request?
As discuss in RATIS-2094, the usage of TransactionContext's
`getStateMachineLogEntry` and `getStateMachineContext` may cause corruptions
duongkame commented on PR #1106:
URL: https://github.com/apache/ratis/pull/1106#issuecomment-2142716387
@szetszwo can you have a look?
--
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
