Github user rvesse commented on a diff in the pull request:
https://github.com/apache/jena/pull/414#discussion_r190532549
--- Diff:
jena-db/jena-dboe-transaction/src/main/java/org/apache/jena/dboe/transaction/txn/TransactionCoordinator.java
---
@@ -142,41 +141,56 @@ public TransactionCoordinator
add(TransactionalComponent elt) {
* @see #add
*/
public TransactionCoordinator remove(TransactionalComponent elt) {
- checkSetup() ;
- synchronized(coordinatorLock) {
- components.remove(elt.getComponentId()) ;
- }
+ checklAllowModification() ;
+ components.remove(elt.getComponentId()) ;
return this ;
}
+
+ /**
+ * Perform modification of this {@code TransaxctionCoordiator} after
it has been started.
--- End diff --
Typo in the `{@code }` here?
---
