set default for destroy method in TbNode interface

rule-engine/rule-engine-api/src/main/java/org/thingsboard/rule/engine/api/TbNode.java

@@ -29,7 +29,7 @@ public interface TbNode {
 
     void onMsg(TbContext ctx, TbMsg msg) throws ExecutionException, InterruptedException, TbNodeException;
 
-    void destroy();
+    default void destroy() {}
 
     default void onPartitionChangeMsg(TbContext ctx, PartitionChangeMsg msg) {}
 

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/action/TbAbstractCustomerActionNode.java

@@ -92,6 +92,9 @@ public abstract class TbAbstractCustomerActionNode<C extends TbAbstractCustomerA
 
     @Override
     public void destroy() {
+        if (customerIdCache != null) {
+            customerIdCache.invalidateAll();
+        }
     }
 
     @Data

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/action/TbAbstractRelationActionNode.java

@@ -88,6 +88,9 @@ public abstract class TbAbstractRelationActionNode<C extends TbAbstractRelationA
 
     @Override
     public void destroy() {
+        if (entityIdCache != null) {
+            entityIdCache.invalidateAll();
+        }
     }
 
     protected ListenableFuture<RelationContainer> processEntityRelationAction(TbContext ctx, TbMsg msg, String relationType) {

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/action/TbCopyAttributesToEntityViewNode.java

@@ -158,7 +158,4 @@ public class TbCopyAttributesToEntityViewNode implements TbNode {
         return CollectionsUtil.contains(keys, attrKey);
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/action/TbMsgCountNode.java

@@ -98,7 +98,4 @@ public class TbMsgCountNode implements TbNode {
         ctx.tellSelf(tickMsg, curDelay);
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/edge/AbstractTbMsgPushNode.java

@@ -71,10 +71,6 @@ public abstract class AbstractTbMsgPushNode<T extends BaseTbMsgPushNodeConfigura
         }
     }
 
-    @Override
-    public void destroy() {
-    }
-
     protected S buildEvent(TbMsg msg, TbContext ctx) {
         String msgType = msg.getType();
         if (DataConstants.ALARM.equals(msgType)) {

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/filter/TbCheckAlarmStatusNode.java

@@ -93,7 +93,4 @@ public class TbCheckAlarmStatusNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/filter/TbCheckMessageNode.java

@@ -68,10 +68,6 @@ public class TbCheckMessageNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-    }
-
     private boolean allKeysData(TbMsg msg) {
         if (!messageNamesList.isEmpty()) {
             Map<String, String> dataMap = dataToMap(msg);

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/filter/TbCheckRelationNode.java

@@ -103,8 +103,4 @@ public class TbCheckRelationNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/filter/TbMsgTypeFilterNode.java

@@ -52,8 +52,4 @@ public class TbMsgTypeFilterNode implements TbNode {
         ctx.tellNext(msg, config.getMessageTypes().contains(msg.getType()) ? "True" : "False");
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/filter/TbMsgTypeSwitchNode.java

@@ -117,8 +117,4 @@ public class TbMsgTypeSwitchNode implements TbNode {
         ctx.tellNext(msg, relationType);
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/filter/TbOriginatorTypeFilterNode.java

@@ -51,8 +51,4 @@ public class TbOriginatorTypeFilterNode implements TbNode {
         ctx.tellNext(msg, config.getOriginatorTypes().contains(originatorType) ? "True" : "False");
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/filter/TbOriginatorTypeSwitchNode.java

@@ -90,8 +90,4 @@ public class TbOriginatorTypeSwitchNode implements TbNode {
         ctx.tellNext(msg, relationType);
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/flow/TbAckNode.java

@@ -51,7 +51,4 @@ public class TbAckNode implements TbNode {
         ctx.tellSuccess(msg);
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/flow/TbCheckpointNode.java

@@ -54,7 +54,4 @@ public class TbCheckpointNode implements TbNode {
         ctx.enqueueForTellNext(msg, queueName, TbRelationTypes.SUCCESS, () -> ctx.ack(msg), error -> ctx.tellFailure(msg, error));
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/flow/TbRuleChainInputNode.java

@@ -63,7 +63,4 @@ public class TbRuleChainInputNode implements TbNode {
         ctx.input(msg, ruleChainId);
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/flow/TbRuleChainOutputNode.java

@@ -51,7 +51,4 @@ public class TbRuleChainOutputNode implements TbNode {
         ctx.output(msg, ctx.getSelf().getName());
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/geo/AbstractGeofencingNode.java

@@ -148,9 +148,4 @@ public abstract class AbstractGeofencingNode<T extends TbGpsGeofencingFilterNode
         return value;
     }
 
-    @Override
-    public void destroy() {
-
-    }
-
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/mail/TbMsgToEmailNode.java

@@ -110,8 +110,4 @@ public class TbMsgToEmailNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/mail/TbSendEmailNode.java

@@ -107,10 +107,6 @@ public class TbSendEmailNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-    }
-
     private JavaMailSenderImpl createMailSender() {
         JavaMailSenderImpl mailSender = new JavaMailSenderImpl();
         mailSender.setHost(this.config.getSmtpHost());

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbAbstractGetAttributesNode.java

@@ -79,10 +79,6 @@ public abstract class TbAbstractGetAttributesNode<C extends TbGetAttributesNodeC
         }
     }
 
-    @Override
-    public void destroy() {
-    }
-
     protected abstract ListenableFuture<T> findEntityIdAsync(TbContext ctx, TbMsg msg);
 
     private void safePutAttributes(TbContext ctx, TbMsg msg, T entityId) {

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbAbstractGetEntityDetailsNode.java

@@ -63,10 +63,6 @@ public abstract class TbAbstractGetEntityDetailsNode<C extends TbAbstractGetEnti
                 t -> ctx.tellFailure(msg, t), ctx.getDbCallbackExecutor());
     }
 
-    @Override
-    public void destroy() {
-    }
-
     protected abstract C loadGetEntityDetailsNodeConfiguration(TbNodeConfiguration configuration) throws TbNodeException;
 
     protected abstract ListenableFuture<TbMsg> getDetails(TbContext ctx, TbMsg msg);

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbEntityGetAttrNode.java

@@ -100,11 +100,6 @@ public abstract class TbEntityGetAttrNode<T extends EntityId> implements TbNode
         ctx.tellSuccess(msg);
     }
 
-    @Override
-    public void destroy() {
-
-    }
-
     protected abstract ListenableFuture<T> findEntityAsync(TbContext ctx, EntityId originator);
 
     public void setConfig(TbGetEntityAttrNodeConfiguration config) {

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbGetOriginatorFieldsNode.java

@@ -80,8 +80,4 @@ public class TbGetOriginatorFieldsNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbGetTelemetryNode.java

@@ -125,10 +125,6 @@ public class TbGetTelemetryNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-    }
-
     private List<ReadTsKvQuery> buildQueries(TbMsg msg, List<String> keys) {
         final Interval interval = getInterval(msg);
         final long aggIntervalStep = Aggregation.NONE.equals(aggregation) ? 1 :

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/rpc/TbSendRPCReplyNode.java

@@ -70,8 +70,4 @@ public class TbSendRPCReplyNode implements TbNode {
         }
     }
 
-    @Override
-    public void destroy() {
-    }
-
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/telemetry/TbMsgAttributesNode.java

@@ -75,8 +75,4 @@ public class TbMsgAttributesNode implements TbNode {
         );
     }
 
-    @Override
-    public void destroy() {
-    }
-
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/transaction/TbSynchronizationBeginNode.java

@@ -49,8 +49,4 @@ public class TbSynchronizationBeginNode implements TbNode {
         ctx.tellSuccess(msg);
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/transaction/TbSynchronizationEndNode.java

@@ -48,7 +48,4 @@ public class TbSynchronizationEndNode implements TbNode {
         ctx.tellSuccess(msg);
     }
 
-    @Override
-    public void destroy() {
-    }
 }

rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/transform/TbChangeOriginatorNode.java

@@ -108,8 +108,4 @@ public class TbChangeOriginatorNode extends TbAbstractTransformNode {
 
     }
 
-    @Override
-    public void destroy() {
-
-    }
 }