Revision 3: Use TemplateUtils.newST to prevent memory leaks

Author: YarivHashaiComet

apps/opik-backend/src/main/java/com/comet/opik/domain/SpanDAO.java

@@ -2335,7 +2335,7 @@ public Mono<Void> bulkUpdate(@NonNull Set<UUID> ids, @NonNull SpanUpdate update,
     }
 
     private ST newBulkUpdateTemplate(SpanUpdate spanUpdate, String sql, boolean mergeTags) {
-        var template = new ST(sql);
+        var template = TemplateUtils.newST(sql);
 
         if (StringUtils.isNotBlank(spanUpdate.name())) {
             template.add("name", spanUpdate.name());

apps/opik-backend/src/main/java/com/comet/opik/domain/TraceDAO.java

@@ -3600,7 +3600,7 @@ public Mono<Void> bulkUpdate(@NonNull Set<UUID> ids, @NonNull TraceUpdate update
     }
 
     private ST newBulkUpdateTemplate(TraceUpdate traceUpdate, String sql, boolean mergeTags) {
-        var template = new ST(sql);
+        var template = TemplateUtils.newST(sql);
 
         if (StringUtils.isNotBlank(traceUpdate.name())) {
             template.add("name", traceUpdate.name());

apps/opik-backend/src/main/java/com/comet/opik/domain/threads/TraceThreadDAO.java

@@ -651,7 +651,7 @@ public Mono<Void> bulkUpdate(@NonNull List<UUID> ids, @NonNull TraceThreadUpdate
     }
 
     private ST newBulkUpdateTemplate(TraceThreadUpdate update, String sql, boolean mergeTags) {
-        var template = new ST(sql);
+        var template = TemplateUtils.newST(sql);
 
         Optional.ofNullable(update.tags())
                 .ifPresent(tags -> {