ensure new transactions and stamps are only created if they don't alr… (

#17)

* ensure new transactions and stamps are only created if they don't already exist

* composer transaction to local variable

* update tinkar-core and eclipse-collections versions

* fix tinkar core version typo

pom.xml

@@ -56,14 +56,14 @@
     </organization>
 
     <properties>
-        <tinkar-core.version>1.72.0</tinkar-core.version>
+        <tinkar-core.version>1.74.0</tinkar-core.version>
         <slf4j.version>2.0.13</slf4j.version>
         <log4j.version>3.0.0-alpha1</log4j.version>
         <tinkar-starter-data.version>1.0.0</tinkar-starter-data.version>
 
         <!-- JPMS Dependencies -->
         <tinkar-jpms-deps.groupId>dev.ikm.jpms</tinkar-jpms-deps.groupId>
-        <eclipse-collections.version>11.1.0-r12</eclipse-collections.version>
+        <eclipse-collections.version>11.1.0-r13</eclipse-collections.version>
         <eclipse-collections-api.version>11.1.0-r11</eclipse-collections-api.version>
     </properties>
 

src/main/java/dev/ikm/tinkar/composer/Composer.java

@@ -27,7 +27,6 @@
 public class Composer {
     private final Map<UUID, Session> composerSessionCache = new HashMap<>();
     private StampEntity stampEntity;
-    private Transaction transaction;
     private final String name;
 
     public Composer(String name) {
@@ -54,10 +53,12 @@ public Composer(String name) {
      * @see State
      */
     public Session open(State status, long time, Concept author, Concept module, Concept path) {
-        this.transaction = new Transaction(name);
-        this.stampEntity = transaction.getStamp(status, time, author.publicId(), module.publicId(), path.publicId());
         UUID sessionKey = keyValue(status, time, author, module, path);
-        composerSessionCache.computeIfAbsent(sessionKey, (key) -> new Session(transaction, stampEntity, sessionKey));
+        composerSessionCache.computeIfAbsent(sessionKey, (key) -> {
+            Transaction transaction = new Transaction(name);
+            this.stampEntity = transaction.getStamp(status, time, author.publicId(), module.publicId(), path.publicId());
+            return new Session(transaction, stampEntity, sessionKey);
+        });
         return composerSessionCache.get(sessionKey);
     }
 
@@ -80,10 +81,12 @@ public Session open(State status, long time, Concept author, Concept module, Con
      * @see State
      */
     public Session open(State status, Concept author, Concept module, Concept path) {
-        this.transaction = new Transaction(name);
-        this.stampEntity = transaction.getStamp(status, author, module, path);
-        UUID sessionKey = keyValue(status, transaction.commitTime(), author, module, path);
-        composerSessionCache.computeIfAbsent(sessionKey, (key) -> new Session(transaction, stampEntity, sessionKey));
+        UUID sessionKey = keyValue(status, Long.MAX_VALUE, author, module, path);
+        composerSessionCache.computeIfAbsent(sessionKey, (key) -> {
+            Transaction transaction = new Transaction(name);
+            this.stampEntity = transaction.getStamp(status, author, module, path);
+            return new Session(transaction, stampEntity, sessionKey);
+        });
         return composerSessionCache.get(sessionKey);
     }
 

src/test/java/module-info.java

@@ -28,8 +28,6 @@
     requires org.junit.jupiter.api;
     requires dev.ikm.jpms.eclipse.collections.api;
 
-
-
     exports dev.ikm.tinkar.composer.test;
     exports dev.ikm.tinkar.composer.test.template;