Fix CsvimIT execution stability - improve files watching reset (#4561)

* reset synchronization watcher when starting synch processing Prevent resetting the watcher after the sync completion since this way it leads to skipped file modifications which occur during long synchronization executions. Signed-off-by: Iliyan Velichkov <velichkov.iliyan@gmail.com> * reduce timeout Signed-off-by: Iliyan Velichkov <velichkov.iliyan@gmail.com> --------- Signed-off-by: Iliyan Velichkov <velichkov.iliyan@gmail.com>
eclipse-dirigible · Jan 13, 2025 · 6e7c747 · 6e7c747
1 parent b8188d7
commit 6e7c747
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.../org/eclipse/dirigible/components/initializers/synchronizer/SynchronizationProcessor.java b/.../org/eclipse/dirigible/components/initializers/synchronizer/SynchronizationProcessor.java
@@ -167,6 +167,7 @@ public void processSynchronizers() {
         }
 
         processing.set(true);
+        synchronizationWatcher.reset();
 
         try {
 
@@ -397,7 +398,6 @@ public void processSynchronizers() {
             definitions.clear();
             artefacts.clear();
 
-            synchronizationWatcher.reset();
             initialized.set(true);
             processing.set(false);
         }

diff --git a/...-integrations/src/test/java/org/eclipse/dirigible/integration/tests/api/java/CsvimIT.java b/...-integrations/src/test/java/org/eclipse/dirigible/integration/tests/api/java/CsvimIT.java
@@ -120,7 +120,7 @@ private void createUndefinedTable() {
     }
 
     private void verifyDataInTable(String tableName, List<Reader> expectedReaders) {
-        await().atMost(60, TimeUnit.SECONDS)
+        await().atMost(30, TimeUnit.SECONDS)
                .pollInterval(1, TimeUnit.SECONDS)
                .until(() -> {
                    try {