Enable wiping of destroyed object names in CP Subsystem [HZ-3007] #25385

gbarnett-hz · 2023-09-06T16:24:42Z

API that permits the wiping of the names that we track for CP objects that have been destroyed. This wiping enables subsequent objects within the same CP group to use names of previously destroyed CP objects. The wiping is applicable for the following:

LockService
SemaphoreService
CountdownLatchService
AtomicRefService
AtomicLongService
CPMapService with additional PR in EE

Example.

// wipes destroyed objects names for each CP group, per-each CP member of the CP group
instance.getCPSubsystem().getCPSubsystemManagementService().wipeDestroyedObjects();

Notes:
With this new functionality, if a CP object is re-created, its linearizable history can be broken. Not all users of the CP object may be aware that the object has been destroyed and they can observe an inconsistent state of it.
For example:

atomicLong.set(2);
atomicLong.destroy();
instance.getCPSubsystem().getCPSubsystemManagementService().wipeDestroyedObjects();
assertEquals(0, atomicLong.get());

Such behaviour description #17498 (comment)

Fixes #25384

Checklist:

Labels (Team:, Type:, Source:, Module:) and Milestone set
Label Add to Release Notes or Not Release Notes content set
Request reviewers if possible
Send backports/forwardports if fix needs to be applied to past/future releases
New public APIs have @Nonnull/@Nullable annotations
New public APIs have @since tags in Javadoc

...rc/main/java/com/hazelcast/cp/internal/datastructures/spi/atomic/RaftAtomicValueService.java

hz-devops-test · 2023-09-06T16:29:49Z

The job Hazelcast-pr-compiler of your PR failed. (Hazelcast internal details: build log, artifacts).
Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log file

---------ERRORS-----------
--------------------------
[ERROR] /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-compiler/hazelcast/src/test/java/com/hazelcast/cp/internal/WipeDestroyedObjectsTest.java:1: Line does not match expected header line of '/*'. [Header]
--------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.3.0:checkstyle (default) on project hazelcast: An error has occurred in Checkstyle report generation. Failed during checkstyle execution: There is 1 error reported by Checkstyle 9.3 with /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-compiler/checkstyle/checkstyle.xml ruleset. -> [Help 1]
--------------------------
[ERROR] 
--------------------------
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
--------------------------
[ERROR] 
--------------------------
[ERROR] For more information about the errors and possible solutions, please read the following articles:
--------------------------
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
--------------------------
[ERROR] 
--------------------------
[ERROR] After correcting the problems, you can resume the build with the command
--------------------------
[ERROR]   mvn  -rf :hazelcast
--------------------------

hz-devops-test · 2023-09-06T16:29:55Z

The job Hazelcast-pr-builder of your PR failed. (Hazelcast internal details: build log, artifacts).
Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log file

---------ERRORS-----------
--------------------------
[ERROR] /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-builder/hazelcast/src/test/java/com/hazelcast/cp/internal/WipeDestroyedObjectsTest.java:1: Line does not match expected header line of '/*'. [Header]
--------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.3.0:checkstyle (default) on project hazelcast: An error has occurred in Checkstyle report generation. Failed during checkstyle execution: There is 1 error reported by Checkstyle 9.3 with /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-builder/checkstyle/checkstyle.xml ruleset. -> [Help 1]
--------------------------
[ERROR] 
--------------------------
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
--------------------------
[ERROR] 
--------------------------
[ERROR] For more information about the errors and possible solutions, please read the following articles:
--------------------------
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
--------------------------
[ERROR] 
--------------------------
[ERROR] After correcting the problems, you can resume the build with the command
--------------------------
[ERROR]   mvn  -rf :hazelcast
--------------------------

hazelcast/src/main/java/com/hazelcast/cp/internal/RaftService.java

gbarnett-hz · 2023-09-06T16:35:29Z

Leaving in DRAFT state for now -- I think we should be careful with this until other bits are progressed.

arodionov

Thanks for the PR, it looks good, but there are a couple of comments

hazelcast/src/main/java/com/hazelcast/cp/CPSubsystemManagementService.java

hazelcast/src/main/java/com/hazelcast/cp/internal/RaftService.java

...rc/main/java/com/hazelcast/cp/internal/datastructures/spi/atomic/RaftAtomicValueService.java

hazelcast/src/main/java/com/hazelcast/cp/internal/raftop/metadata/WipeDestroyedObjectsOp.java

hazelcast/src/main/java/com/hazelcast/cp/internal/RaftService.java

hazelcast/src/test/java/com/hazelcast/cp/internal/WipeDestroyedObjectsTest.java

arodionov · 2023-09-07T08:18:48Z

@gbarnett-hz @vbekiaris I think it will be also Okay if, in addition to
cpSubsystemManagementService.wipeDestroyedObjects();
that wipes destroyed object names in all CP groups, we create an endpoint to wipe a particular group:
cpSubsystemManagementService.wipeDestroyedObjects(CPGroupID cpGroupID);
It also makes sense for granularity and testability reasons.
WDYT?

gbarnett-hz · 2023-09-07T08:30:25Z

@arodionov I had a preference more for the API wipeDestroyedObjects(CPGroupId) but it seemed that on the previous communication there was a requirement for a single simple solution, hence wipeDestroyedObjects(). As the latter makes use of the former internally I don't think there's an issue exposing this unless we don't want to clutter the management API which seems to be pretty concise and focused from what I see...leading to maybe just exposing wipeDestroyedObjects().

arodionov · 2023-09-07T09:12:40Z

@gbarnett-hz Yes, you are right. My previous comments were not to expose additional APIs as I thought we would not handle each CP group separately. But while we're at it, it might make sense to also expose this method via a public API.

vbekiaris

Added a couple comments and a few thumbs-up in previous comments.
Let's keep this PR on the side and revisit once we have a solid solution for CP object reuse.
thanks @gbarnett-hz

hazelcast/src/main/java/com/hazelcast/cp/internal/CPSubsystemImpl.java

hazelcast/src/main/java/com/hazelcast/cp/internal/raftop/metadata/WipeDestroyedObjectsOp.java

hazelcast/src/test/java/com/hazelcast/cp/internal/WipeDestroyedObjectsTest.java

hz-devops-test · 2023-09-08T17:40:58Z

The job Hazelcast-pr-builder of your PR failed. (Hazelcast internal details: build log, artifacts).
Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log file

---------ERRORS----------- -------------------------- [ERROR] Tests run: 17, Failures: 1, Errors: 0, Skipped: 1, Time elapsed: 222.6 s <<< FAILURE! -- in com.hazelcast.jet.kinesis.KinesisIntegrationTest -------------------------- [ERROR] com.hazelcast.jet.kinesis.KinesisIntegrationTest.restart_dynamicStream_graceful -- Time elapsed: 125.7 s <<< FAILURE! -------------------------- [ERROR] Failures: -------------------------- [ERROR] KinesisIntegrationTest.restart_dynamicStream_graceful:387->restart_dynamicStream:425->assertMessages:637->AbstractKinesisTest.assertMessages:163->HazelcastTestSupport.assertTrueEventually:1269->HazelcastTestSupport.assertTrueEventually:1165->AbstractKinesisTest.lambda$assertMessages$3:180 Messages for key 247 differ! -------------------------- [ERROR] Tests run: 26, Failures: 1, Errors: 0, Skipped: 3 -------------------------- [ERROR] There are test failures. -------------------------- -------------------------- -------TEST FAILURE------- -------------------------- [INFO] Results: [INFO] [ERROR] Failures: [ERROR] KinesisIntegrationTest.restart_dynamicStream_graceful:387->restart_dynamicStream:425->assertMessages:637->AbstractKinesisTest.assertMessages:163->HazelcastTestSupport.assertTrueEventually:1269->HazelcastTestSupport.assertTrueEventually:1165->AbstractKinesisTest.lambda$assertMessages$3:180 Messages for key 247 differ! expected: 10 actual: 1506 expected:<[247: msg 000000247, 247: msg 000000497, 247: msg 000000747, 247: msg 000000997, 247: msg 000001247, 247: msg 000001497, 247: msg 000001747, 247: msg 000001997, 247: msg 000002247, 247: msg 000002497]> but was:<[247: msg 000000247, 247: msg 000000497, 247: msg 000000747, 247: msg 000000997, 247: msg 000001247, 247: msg 000001497, 247: msg 000001747, 247: msg 000001997, 247: msg 000002247, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497, 247: msg 000002497]> [INFO] [ERROR] Tests run: 26, Failures: 1, Errors: 0, Skipped: 3 [INFO]

[ERROR] There are test failures.

arodionov · 2023-09-08T19:49:08Z

run-lab-run

hazelcast/src/main/java/com/hazelcast/cp/CPSubsystemManagementService.java

hazelcast/src/main/java/com/hazelcast/cp/internal/CPSubsystemImpl.java

hazelcast/src/main/java/com/hazelcast/cp/internal/raftop/metadata/WipeDestroyedObjectsOp.java

...rc/main/java/com/hazelcast/cp/internal/datastructures/spi/atomic/RaftAtomicValueService.java

hazelcast/src/test/java/com/hazelcast/cp/internal/WipeDestroyedObjectsTest.java

hazelcast/src/main/java/com/hazelcast/cp/internal/raftop/metadata/WipeDestroyedObjectsOp.java

hazelcast/src/main/java/com/hazelcast/cp/internal/CPSubsystemImpl.java

hz-devops-test · 2023-09-15T17:40:37Z

The job Hazelcast-pr-builder of your PR failed. (Hazelcast internal details: build log, artifacts).
Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log file

---------ERRORS-----------
--------------------------
[ERROR] Tests run: 28, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 224.4 s <<< FAILURE! -- in com.hazelcast.jet.kafka.impl.StreamKafkaPTest
--------------------------
[ERROR] com.hazelcast.jet.kafka.impl.StreamKafkaPTest.when_processingGuaranteeNoneWithConsumerGroup_then_continueFromLastReadMessageAfterJobRestart -- Time elapsed: 123.6 s <<< FAILURE!
--------------------------
[ERROR] Failures: 
--------------------------
[ERROR]   StreamKafkaPTest.when_processingGuaranteeNoneWithConsumerGroup_then_continueFromLastReadMessageAfterJobRestart:266->testWithJobRestart:345->HazelcastTestSupport.assertTrueEventually:1269->HazelcastTestSupport.assertTrueEventually:1165->lambda$testWithJobRestart$4:345 expected:<200> but was:<198>
--------------------------
[ERROR] Tests run: 51, Failures: 1, Errors: 0, Skipped: 0
--------------------------
[ERROR] There are test failures.
--------------------------
--------------------------
-------TEST FAILURE-------
--------------------------
[INFO] Results:
[INFO] 
[ERROR] Failures: 
[ERROR]   StreamKafkaPTest.when_processingGuaranteeNoneWithConsumerGroup_then_continueFromLastReadMessageAfterJobRestart:266->testWithJobRestart:345->HazelcastTestSupport.assertTrueEventually:1269->HazelcastTestSupport.assertTrueEventually:1165->lambda$testWithJobRestart$4:345 expected:<200> but was:<198>
[INFO] 
[ERROR] Tests run: 51, Failures: 1, Errors: 0, Skipped: 0
[INFO] 
[ERROR] There are test failures.

gbarnett-hz · 2023-09-18T08:20:03Z

run-lab-run

- remove IndeterminateOperationStateAware - previous has same semantics with it removed - require cluster version >= 5.4 for wipeDestroyedObjects

- ResourceRegistryAccessor - NodeEngine#getServices(...) for determining services to clear

- compat comment

arodionov · 2023-11-09T17:18:52Z

@gbarnett-hz @vbekiaris I've refactored a current PR to be able to support the CPMap.
Please take a look

gbarnett-hz · 2023-11-10T14:21:34Z

@arodionov LGTM

devOpsHazelcast · 2024-02-16T22:01:38Z

PR closed by Hazelcast automation as no activity (>3 months). Please reopen with comments, if necessary. Thank you for using Hazelcast and your valuable contributions

gbarnett-hz added Type: Enhancement Team: Core Source: Internal PR or issue was opened by an employee Module: CP Subsystem labels Sep 6, 2023

gbarnett-hz self-assigned this Sep 6, 2023

gbarnett-hz added this to the 5.4.0 milestone Sep 6, 2023

gbarnett-hz marked this pull request as draft September 6, 2023 16:25

gbarnett-hz commented Sep 6, 2023

View reviewed changes

...rc/main/java/com/hazelcast/cp/internal/datastructures/spi/atomic/RaftAtomicValueService.java Outdated Show resolved Hide resolved

gbarnett-hz requested review from arodionov and vbekiaris September 6, 2023 16:32

gbarnett-hz commented Sep 6, 2023

View reviewed changes

hazelcast/src/main/java/com/hazelcast/cp/internal/RaftService.java Outdated Show resolved Hide resolved

arodionov added the Add to Release Notes label Sep 6, 2023

arodionov reviewed Sep 7, 2023

View reviewed changes

vbekiaris reviewed Sep 8, 2023

View reviewed changes

gbarnett-hz requested review from vbekiaris and arodionov September 11, 2023 14:07

arodionov reviewed Sep 11, 2023

View reviewed changes

gbarnett-hz requested a review from arodionov September 12, 2023 08:10

arodionov approved these changes Sep 13, 2023

View reviewed changes

vbekiaris reviewed Sep 14, 2023

View reviewed changes

hazelcast/src/main/java/com/hazelcast/cp/internal/CPSubsystemImpl.java Outdated Show resolved Hide resolved

vbekiaris approved these changes Sep 18, 2023

View reviewed changes

gbarnett-hz and others added 12 commits November 9, 2023 15:51

Wipe destroyed objects

411a3e6

Add @SInCE tag

9e4147d

Copyright header on test.

c480714

- restrict removal of destroyedValues to cpGroup of Raft Op

37abc32

- remove IndeterminateOperationStateAware - previous has same semantics with it removed - require cluster version >= 5.4 for wipeDestroyedObjects

- RaftAtomicValueServiceAccessor

2e5c695

- ResourceRegistryAccessor - NodeEngine#getServices(...) for determining services to clear

- Use CompletionStage<Void> for API

4232795

- Re-create CP objects with the same names we just wiped

3891df9

- Javadoc correction

7aff69c

- compat comment

- Move clearDestroyedNames logic to AbstractBlockingService

dd0d2b9

- Simplify tests

2536e95

- Use Cluster service to get Version

38aa28d

Refactor WipeDestroyedObjectsOp to support CPMap from Enterprise part

5ba89ec

arodionov force-pushed the hz-3007 branch from dec631d to 5ba89ec Compare November 9, 2023 16:14

arodionov self-assigned this Nov 9, 2023

arodionov requested a review from vbekiaris November 9, 2023 17:18

gbarnett-hz changed the title ~~Enable wiping of destroyed object names in CP Subsystem~~ Enable wiping of destroyed object names in CP Subsystem [HZ-3007] Nov 10, 2023

devOpsHazelcast closed this Feb 16, 2024

devOpsHazelcast added the Automation: PR auto closed label Feb 16, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Enable wiping of destroyed object names in CP Subsystem [HZ-3007] #25385

Enable wiping of destroyed object names in CP Subsystem [HZ-3007] #25385

gbarnett-hz commented Sep 6, 2023 •

edited by arodionov

hz-devops-test commented Sep 6, 2023

hz-devops-test commented Sep 6, 2023

gbarnett-hz commented Sep 6, 2023

arodionov left a comment

arodionov commented Sep 7, 2023 •

edited

gbarnett-hz commented Sep 7, 2023

arodionov commented Sep 7, 2023 •

edited

vbekiaris left a comment

hz-devops-test commented Sep 8, 2023

arodionov commented Sep 8, 2023

hz-devops-test commented Sep 15, 2023

gbarnett-hz commented Sep 18, 2023

arodionov commented Nov 9, 2023

gbarnett-hz commented Nov 10, 2023

devOpsHazelcast commented Feb 16, 2024

Enable wiping of destroyed object names in CP Subsystem [HZ-3007] #25385

Enable wiping of destroyed object names in CP Subsystem [HZ-3007] #25385

Conversation

gbarnett-hz commented Sep 6, 2023 • edited by arodionov

hz-devops-test commented Sep 6, 2023

hz-devops-test commented Sep 6, 2023

gbarnett-hz commented Sep 6, 2023

arodionov left a comment

Choose a reason for hiding this comment

arodionov commented Sep 7, 2023 • edited

gbarnett-hz commented Sep 7, 2023

arodionov commented Sep 7, 2023 • edited

vbekiaris left a comment

Choose a reason for hiding this comment

hz-devops-test commented Sep 8, 2023

arodionov commented Sep 8, 2023

hz-devops-test commented Sep 15, 2023

gbarnett-hz commented Sep 18, 2023

arodionov commented Nov 9, 2023

gbarnett-hz commented Nov 10, 2023

devOpsHazelcast commented Feb 16, 2024

gbarnett-hz commented Sep 6, 2023 •

edited by arodionov

arodionov commented Sep 7, 2023 •

edited

arodionov commented Sep 7, 2023 •

edited