Cleaned code after migration from Bitbucket

Author: michalkijas

README.md

@@ -1,63 +1,31 @@
-# `@Transactional` Sandbox
+Materials gathered for internal presentation
 
-## Transakcje
+## Transactions
   * https://docs.spring.io/spring/docs/current/spring-framework-reference/data-access.html
     * https://docs.spring.io/spring/docs/current/spring-framework-reference/data-access.html#transaction-declarative-attransactional-settings
   * https://www.baeldung.com/java-transactions
   * https://vladmihalcea.com/?s=transaction&submit=Go
-#### Pitfals  
+
+## Pitfals  
   * [5 common Spring @Transactional pitfalls](https://codete.com/blog/5-common-spring-transactional-pitfalls/)
     * [Checked and Unchecked Exceptions](https://www.codejava.net/java-core/exception/java-checked-and-unchecked-exceptions)
   * [Spring Transactional propagation modes](https://codete.com/blog/spring-transaction-propagation-modes/)
   * https://www.ibm.com/developerworks/java/library/j-ts1/index.html
   * https://www.javacodegeeks.com/2011/12/spring-pitfalls-transactional-tests.html
 
-## Testowanie użycia annotacji `@Transactional` 
+## Testing `@Transactional` 
   * [ArchUnit - dokumentacja](https://www.archunit.org/getting-started)
   * [ArchUnit - gradle plugin](https://github.com/societe-generale/arch-unit-gradle-plugin)
 
-
-## read-only and read-write operation
+## Operation read-only and read-write
   * https://vladmihalcea.com/read-write-read-only-transaction-routing-spring/
   * https://fable.sh/blog/splitting-read-and-write-operations-in-spring-boot/
   * https://blog.pchudzik.com/201911/read-from-replica/
-## disable autocommit
+
+## Disable autocommit
   * https://vladmihalcea.com/why-you-should-always-use-hibernate-connection-provider_disables_autocommit-for-resource-local-jpa-transactions/
   * https://dzone.com/articles/best-performance-practices-for-hibernate-5-and-spr
+
 ## Open session in view
   * https://vladmihalcea.com/the-open-session-in-view-anti-pattern/
   * [The OSIV Anti-Pattern](https://stackoverflow.com/questions/30549489/what-is-this-spring-jpa-open-in-view-true-property-in-spring-boot/48222934#48222934)
-
-## Do zastanowienia się
-* jak działą Transactional(readOnly=true) jeżęli sesja jest otwierana przez OpenSessionInView któ©y chyba domyślnie jest read-write i wymusza commit nawet jeżęli tylko dane są odczytywane
-* zamiana save() -> @Transactional - zastąpienie zapisu Lad i Inerakcji tylko zapisem Lead ? -  zapis encji + kolekcji (lead+interakcje)
-* zakres tansakcji przy imporcie bazy pacjentów - co poszło nie tak
-* różne poziomy izolacji transakcji
-* zdarzenia DDD ???
-* sprawdzić notatkic z szkolenia Kubryńskiego - Hibernate
-
-#### Derby DB Replica
-http://db.apache.org/derby/docs/10.4/adminguide/cadminreplicstartrun.html
-
----
-## Podsumowanie:
- - wstęp do tego czym jest transakcja - nie koniecznie DB
- - Czy można ręczne zarządzać trasakcjami tak jak w EJB
- - Dziedziczenie i ukrywanie użycia annotacji
- - OSIV vs radOnly - komibtowanie zmian po zakonczeniu operacji
- - Rozwinąć opis Propagation.NESTED
- - Synchronizacja z transakcjami - createOrUpdateNotification - ja zapewnić że posiadamy poprawnie zapisany obiekt
-    - REQUIRE_NEW
-    - entityManager/transactionManager .synchronize ???
-
-
-start
-  code
-  code
-  < exception > 
-  code 
-commit / rolback
-
-
-
-https://docstore.mik.ua/orelly/java-ent/ebeans/ch08_02.htm

build.gradle

@@ -53,33 +53,9 @@ dependencies {
         exclude group: "org.codehaus.groovy", module: "groovy-testng"
     }
     testImplementation "org.springframework.boot:spring-boot-starter-test"
-
     testImplementation 'com.tngtech.archunit:archunit:0.14.1'
-//    testImplementation "org.springframework.security:spring-security-test"
 }
 
 test {
     useJUnitPlatform()
 }
-
-// Disabling ArchUnit in `build` phase
-//allprojects {
-//    apply plugin: 'java'
-//    apply plugin: 'com.societegenerale.commons.plugin.gradle.ArchUnitGradlePlugin'
-//    archUnit {
-//        excludedPaths = ["some/package"]
-//        preConfiguredRules = [
-//                "com.societegenerale.commons.plugin.rules.NoInjectedFieldTest",
-//                "com.societegenerale.commons.plugin.rules.NoAutowiredFieldTest",
-//                "com.societegenerale.commons.plugin.rules.NoTestIgnoreWithoutCommentRuleTest",
-//                "com.societegenerale.commons.plugin.rules.NoPrefixForInterfacesRuleTest",
-//                "com.societegenerale.commons.plugin.rules.NoPowerMockRuleTest",
-//                "com.societegenerale.commons.plugin.rules.NoJodaTimeRuleTest",
-//                "com.societegenerale.commons.plugin.rules.NoJunitAssertRuleTest",
-//                "com.societegenerale.commons.plugin.rules.HexagonalArchitectureTest",
-//                "com.societegenerale.commons.plugin.rules.DontReturnNullCollectionTest",
-//                "com.societegenerale.commons.plugin.rules.StringFieldsThatAreActuallyDatesRuleTest"
-//        ]
-//        configurableRules = [configurableRule("com.tngtech.archunit.library.GeneralCodingRules", applyOn("com.example.demo", "main"))]
-//    }
-//}

settings.gradle

@@ -1 +1 @@
-rootProject.name = 'sandbox_transactional'
+rootProject.name = 'presentation_transactional'

src/main/java/com/example/demo/sandbox/TransactionPropagationService.java

@@ -1,7 +1,6 @@
 package com.example.demo.sandbox;
 
 import lombok.RequiredArgsConstructor;
-import org.springframework.transaction.annotation.Propagation;
 import org.springframework.transaction.annotation.Transactional;
 
 @RequiredArgsConstructor
@@ -11,25 +10,6 @@ class TransactionPropagationService {
     private final LeadInteractionService leadInteractionService;
 
 
-//    @Transactional
-//    public void runFirst(String comment, String leadInteractionName) {
-////        runInTransaction1(comment);
-//    }
-//
-//
-//    @Transactional(propagation = Propagation.REQUIRED)
-//    @Transactional(propagation = Propagation.REQUIRES_NEW)
-//    @Transactional(propagation = Propagation.MANDATORY)
-//    @Transactional(propagation = Propagation.SUPPORTS)
-//    @Transactional(propagation = Propagation.NOT_SUPPORTED)
-//    @Transactional(propagation = Propagation.NEVER)
-//    @Transactional(propagation = Propagation.NESTED)
-//    public void runInTransaction1(String comment, String leadInteractionName) {
-//        Lead lead = new Lead();
-//        lead.setComment(comment);
-//    }
-
-
     @Transactional
     public Lead runInTransaction(String comment, String leadInteractionName) {
         Lead lead = new Lead();

src/main/java/com/example/demo/sandbox/VisibilityMethodTransactionService.java

@@ -2,36 +2,12 @@
 
 import lombok.RequiredArgsConstructor;
 import org.springframework.transaction.annotation.Transactional;
-import javax.persistence.EntityManager;
-import javax.persistence.EntityTransaction;
 
 @RequiredArgsConstructor
 class VisibilityMethodTransactionService {
 
     private final LeadRepository leadRepository;
 
-//    private final EntityManager em;
-//
-//    void run() {
-//        EntityTransaction transaction = em.getTransaction();
-//        try {
-//            transaction.begin();
-////        code
-////        code
-////        code
-//            transaction.commit();
-//        } catch (Exception e) {
-//            transaction.rollback();
-//        }
-//    }
-
-
-    @Transactional
-    public Lead createLeadUsingSaveMethod2(String comment) {
-        Lead lead = new Lead();
-        lead.setComment(comment);
-        return lead;
-    }
 
     Lead createLeadUsingSaveMethod(String comment) {
         Lead lead = new Lead();
@@ -43,15 +19,8 @@ Lead createLeadUsingSaveMethod(String comment) {
     public void updateByPublicMethod(Long leadId, String comment) {
         Lead lead = leadRepository.getByIdOrThrow(leadId);
         lead.setComment(comment);
-//        updateByPublicMethod1(...);
     }
 
-//    @Transactional
-//    public void updateByPublicMethod1(Long leadId, String comment) {
-//        Lead lead = leadRepository.getByIdOrThrow(leadId);
-//        lead.setComment(comment);
-//    }
-
     @Transactional
     void updateByPackageMethod(Long leadId, String comment) {
         Lead lead = leadRepository.getByIdOrThrow(leadId);

src/test/java/com/example/demo/SandboxTransactionalApplicationTests.java

@@ -6,8 +6,7 @@
 @SpringBootTest
 class SandboxTransactionalApplicationTests {
 
-	@Test
-	void contextLoads() {
-	}
+    @Test
+    void contextLoads() {}
 
 }