mark clientSpan and traceId nullable + add logs

Author: MykolaYaremchuk

ide-common/src/main/java/org/digma/intellij/plugin/analytics/AnalyticsService.java

@@ -160,16 +160,19 @@ public void sendDebuggerEvent(int eventType,String timestamp) throws AnalyticsSe
 
     public List<GlobalInsight> getGlobalInsights() throws AnalyticsServiceException {
         var env = getCurrentEnvironment();
+        Log.log(LOGGER::debug, "Requesting Global Insights for next environment {}", env);
         return executeCatching(() -> analyticsProviderProxy.getGlobalInsights(new InsightsRequest(env, Collections.emptyList())));
     }
 
     public List<CodeObjectInsight> getInsights(List<String> objectIds) throws AnalyticsServiceException {
         var env = getCurrentEnvironment();
+        Log.log(LOGGER::debug, "Requesting insights for next objectIds {} and next environment {}", objectIds, env);
         return executeCatching(() -> analyticsProviderProxy.getInsights(new InsightsRequest(env, objectIds)));
     }
 
     public List<CodeObjectError> getErrorsOfCodeObject(String codeObjectId) throws AnalyticsServiceException {
         var env = getCurrentEnvironment();
+        Log.log(LOGGER::debug, "Requesting insights for next codeObjectId {} and next environment {}", codeObjectId, env);
         return executeCatching(() -> analyticsProviderProxy.getErrorsOfCodeObject(env, codeObjectId));
     }
 

ide-common/src/main/java/org/digma/intellij/plugin/document/DocumentInfoContainer.java

@@ -85,6 +85,7 @@ private void loadAllInsightsForCurrentDocument() {
             //insights = Collections.emptyList() means there was an error loading insights, usually if the backend is not available.
             //don't log the exception, it was logged in AnalyticsService, keep the log quite because it can happen many times.
             insights = Collections.emptyList();
+            Log.log(LOGGER::debug, "Cannot get insights with ids: {}. Because: {}", objectIds, e.getMessage());
         }
 
         try {

model/src/main/kotlin/org/digma/intellij/plugin/model/rest/insights/HighlyOccurringSpanInfo.kt

@@ -11,8 +11,8 @@ data class HighlyOccurringSpanInfo
 constructor(
         val occurrences: Double,
         val internalSpan: SpanInfo?,
-        val clientSpan: SpanInfo,
-        val traceId: String,
+        val clientSpan: SpanInfo?,
+        val traceId: String?,
         val duration: Duration,
         val fraction: Double,
 )

src/main/kotlin/org/digma/intellij/plugin/ui/list/insights/EPNPlusSpansPanel.kt

@@ -41,7 +41,10 @@ fun ePNPlusSpansPanel(project: Project, insight: EPNPlusSpansInsight): JPanel {
 }
 
 private fun getSQLRowPanel(insight: EPNPlusSpansInsight): JPanel {
-    val sqlStatement = insight.spans.first().clientSpan.displayName
+    var sqlStatement = insight.spans.first().clientSpan?.displayName
+    if (sqlStatement == null) {
+        sqlStatement = insight.spans.first().internalSpan?.displayName
+    }
     val normalizedDisplayName = StringUtils.normalizeSpace(sqlStatement)
     val displayNameLabel = JBLabel(normalizedDisplayName, SwingConstants.TRAILING)
     displayNameLabel.toolTipText = asHtml(sqlStatement)