navigation discovery

Author: shalom938

ide-common/src/main/kotlin/org/digma/intellij/plugin/discovery/model/EndpointLocation.kt

@@ -4,4 +4,11 @@ import com.intellij.openapi.vfs.VirtualFile
 
 class EndpointLocation(val file: VirtualFile, val endpointId: String, val offset: Int, val methodCodeObjectId: String) {
     fun isAlive(): Boolean = file.isValid
+
+
+
+    override fun toString(): String {
+        return "EndpointLocation(file=$file, endpointId='$endpointId', offset=$offset, methodCodeObjectId='$methodCodeObjectId')"
+    }
+
 }

ide-common/src/main/kotlin/org/digma/intellij/plugin/discovery/model/SpanLocation.kt

@@ -4,4 +4,10 @@ import com.intellij.openapi.vfs.VirtualFile
 
 class SpanLocation(val file: VirtualFile, val offset: Int, val methodCodeObjectId: String) {
     fun isAlive(): Boolean = file.isValid
+
+    override fun toString(): String {
+        return "SpanLocation(file=$file, offset=$offset, methodCodeObjectId='$methodCodeObjectId')"
+    }
+
+
 }

jvm-common/src/main/kotlin/org/digma/intellij/plugin/idea/navigation/JvmEndpointNavigationProvider.kt

@@ -60,10 +60,21 @@ internal class JvmEndpointNavigationProvider(private val project: Project) {
 
 
     suspend fun maintenance() {
+
+        if(logger.isTraceEnabled){
+            Log.trace(logger, "starting maintenance, current endpoint location count ${endpointsMap.size}")
+            endpointsMap.forEach { (endpointId, endpointLocations) ->
+                Log.trace(logger, "endpoint location for endpoint {}: [{}]", endpointId, endpointLocations)
+            }
+        }
+
         maintenanceLock.withLock {
             val toRemove = endpointsMap.filter { (_, endpointLocations) ->
                 !endpointLocations.any { it.isAlive() } || !endpointLocations.any { hasIndex(project, it.file) }
             }.keys
+            if (toRemove.isEmpty()) {
+                return@withLock
+            }
             Log.trace(logger, "maintenance removing endpoints {}", toRemove)
             endpointsMap.entries.removeIf { toRemove.contains(it.key) }
         }

jvm-common/src/main/kotlin/org/digma/intellij/plugin/idea/navigation/JvmNavigationDiscoveryManager.kt

@@ -219,7 +219,8 @@ class JvmNavigationDiscoveryManager(private val project: Project, private val cs
 
 
     private fun launchMaintenanceJob() {
-        cs.launchWhileActiveWithErrorReporting(1.minutes, 30.seconds, "${this::class.java.simpleName}.MaintenanceTask", logger) {
+        cs.launchWhileActiveWithErrorReporting(1.minutes, 1.minutes, "${this::class.java.simpleName}.MaintenanceTask", logger) {
+            DumbService.getInstance(project).waitForSmartMode()
             JvmSpanNavigationProvider.getInstance(project).maintenance()
             JvmEndpointNavigationProvider.getInstance(project).maintenance()
         }

jvm-common/src/main/kotlin/org/digma/intellij/plugin/idea/navigation/JvmSpanNavigationProvider.kt

@@ -70,8 +70,19 @@ internal class JvmSpanNavigationProvider(private val project: Project) {
     }
 
     suspend fun maintenance() {
+
+        if(logger.isTraceEnabled){
+            Log.trace(logger, "starting maintenance, current span location count ${spanLocations.size}")
+            spanLocations.forEach { (spanId, spanLocation) ->
+                Log.trace(logger, "span location for span {} [{}]", spanId, spanLocation)
+            }
+        }
+
         maintenanceLock.withLock {
             val toRemove = spanLocations.filter { (_, spanLocation) -> !spanLocation.isAlive() || !hasIndex(project, spanLocation.file) }.keys
+            if (toRemove.isEmpty()) {
+                return@withLock
+            }
             Log.trace(logger, "maintenance removing spans {}", toRemove)
             spanLocations.entries.removeIf { toRemove.contains(it.key) }
         }