JS Stats

@@ -33,6 +33,7 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.context.annotation.Lazy;
 import org.springframework.data.redis.core.RedisTemplate;
+import org.springframework.scheduling.annotation.Scheduled;
 import org.springframework.stereotype.Component;
 import org.thingsboard.rule.engine.api.MailService;
 import org.thingsboard.rule.engine.api.RuleChainTransactionService;
@@ -89,6 +90,7 @@ import java.io.StringWriter;
 import java.util.Optional;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ConcurrentMap;
+import java.util.concurrent.atomic.AtomicInteger;
 
 @Slf4j
 @Component
@@ -286,6 +288,21 @@ public class ActorSystemContext {
     @Getter
     private long statisticsPersistFrequency;
 
+    @Getter
+    private final AtomicInteger jsInvokeRequestsCount = new AtomicInteger(0);
+    @Getter
+    private final AtomicInteger jsInvokeResponsesCount = new AtomicInteger(0);
+    @Getter
+    private final AtomicInteger jsInvokeFailuresCount = new AtomicInteger(0);
+
+    @Scheduled(fixedDelayString = "${js.remote.stats.print_interval_ms}")
+    public void printStats() {
+        if (statisticsEnabled) {
+            log.info("Rule Engine JS Invoke Stats: requests [{}] responses [{}] failures [{}]",
+                    jsInvokeRequestsCount.getAndSet(0), jsInvokeResponsesCount.getAndSet(0), jsInvokeFailuresCount.getAndSet(0));
+        }
+    }
+
     @Value("${actors.tenant.create_components_on_init}")
     @Getter
     private boolean tenantComponentsInitEnabled;

@@ -230,6 +230,27 @@ class DefaultTbContext implements TbContext {
     }
 
     @Override
+    public void logJsEvalRequest() {
+        if (mainCtx.isStatisticsEnabled()) {
+            mainCtx.getJsInvokeRequestsCount().incrementAndGet();
+        }
+    }
+
+    @Override
+    public void logJsEvalResponse() {
+        if (mainCtx.isStatisticsEnabled()) {
+            mainCtx.getJsInvokeResponsesCount().incrementAndGet();
+        }
+    }
+
+    @Override
+    public void logJsEvalFailure() {
+        if (mainCtx.isStatisticsEnabled()) {
+            mainCtx.getJsInvokeFailuresCount().incrementAndGet();
+        }
+    }
+
+    @Override
     public String getNodeId() {
         return mainCtx.getNodeIdProvider().getNodeId();
     }

@@ -131,15 +131,21 @@ public class ConsistentClusterRoutingService implements ClusterRoutingService {
     private void addNode(ServerInstance instance) {
         for (int i = 0; i < virtualNodesSize; i++) {
             circles[instance.getServerAddress().getServerType().ordinal()].put(hash(instance, i).asLong(), instance);
+//            circles[instance.getServerAddress().getServerType().ordinal()].put(classic(instance, i), instance);
         }
     }
 
     private void removeNode(ServerInstance instance) {
         for (int i = 0; i < virtualNodesSize; i++) {
             circles[instance.getServerAddress().getServerType().ordinal()].remove(hash(instance, i).asLong());
+//            circles[instance.getServerAddress().getServerType().ordinal()].remove(classic(instance, i));
         }
     }
 
+    private long classic(ServerInstance instance, int i) {
+        return (instance.getHost() + instance.getPort() + i).hashCode() * (Long.MAX_VALUE / Integer.MAX_VALUE);
+    }
+
     private HashCode hash(ServerInstance instance, int i) {
         return hashFunction.newHasher().putString(instance.getHost(), MiscUtils.UTF8).putInt(instance.getPort()).putInt(i).hash();
     }

@@ -22,6 +22,7 @@ import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
+import org.springframework.scheduling.annotation.Scheduled;
 import org.springframework.stereotype.Service;
 import org.thingsboard.server.gen.js.JsInvokeProtos;
 import org.thingsboard.server.kafka.TBKafkaConsumerTemplate;
@@ -29,13 +30,14 @@ import org.thingsboard.server.kafka.TBKafkaProducerTemplate;
 import org.thingsboard.server.kafka.TbKafkaRequestTemplate;
 import org.thingsboard.server.kafka.TbKafkaSettings;
 import org.thingsboard.server.kafka.TbNodeIdProvider;
-import org.thingsboard.server.service.cluster.discovery.DiscoveryService;
 
 import javax.annotation.PostConstruct;
 import javax.annotation.PreDestroy;
 import java.util.Map;
 import java.util.UUID;
 import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.atomic.AtomicInteger;
+import java.util.concurrent.atomic.AtomicLong;
 
 @Slf4j
 @ConditionalOnProperty(prefix = "js", value = "evaluator", havingValue = "remote", matchIfMissing = true)
@@ -70,6 +72,25 @@ public class RemoteJsInvokeService extends AbstractJsInvokeService {
     @Value("${js.remote.max_errors}")
     private int maxErrors;
 
+    @Value("${js.remote.stats.enabled:false}")
+    private boolean statsEnabled;
+
+    private final AtomicInteger kafkaPushedMsgs = new AtomicInteger(0);
+    private final AtomicInteger kafkaInvokeMsgs = new AtomicInteger(0);
+    private final AtomicInteger kafkaEvalMsgs = new AtomicInteger(0);
+    private final AtomicInteger kafkaFailedMsgs = new AtomicInteger(0);
+
+    @Scheduled(fixedDelayString = "${js.remote.stats.print_interval_ms}")
+    public void printStats() {
+        if (statsEnabled) {
+            int invokeMsgs = kafkaInvokeMsgs.getAndSet(0);
+            int evalMsgs = kafkaEvalMsgs.getAndSet(0);
+            int failed = kafkaFailedMsgs.getAndSet(0);
+            log.info("Kafka JS Invoke Stats: pushed [{}] received [{}] invoke [{}] eval [{}] failed [{}]",
+                    kafkaPushedMsgs.getAndSet(0), invokeMsgs + evalMsgs, invokeMsgs, evalMsgs, failed);
+        }
+    }
+
     private TbKafkaRequestTemplate<JsInvokeProtos.RemoteJsRequest, JsInvokeProtos.RemoteJsResponse> kafkaTemplate;
     private Map<UUID, String> scriptIdToBodysMap = new ConcurrentHashMap<>();
 
@@ -139,14 +160,17 @@ public class RemoteJsInvokeService extends AbstractJsInvokeService {
 
         log.trace("Post compile request for scriptId [{}]", scriptId);
         ListenableFuture<JsInvokeProtos.RemoteJsResponse> future = kafkaTemplate.post(scriptId.toString(), jsRequestWrapper);
+        kafkaPushedMsgs.incrementAndGet();
         return Futures.transform(future, response -> {
             JsInvokeProtos.JsCompileResponse compilationResult = response.getCompileResponse();
             UUID compiledScriptId = new UUID(compilationResult.getScriptIdMSB(), compilationResult.getScriptIdLSB());
             if (compilationResult.getSuccess()) {
+                kafkaEvalMsgs.incrementAndGet();
                 scriptIdToNameMap.put(scriptId, functionName);
                 scriptIdToBodysMap.put(scriptId, scriptBody);
                 return compiledScriptId;
             } else {
+                kafkaFailedMsgs.incrementAndGet();
                 log.debug("[{}] Failed to compile script due to [{}]: {}", compiledScriptId, compilationResult.getErrorCode().name(), compilationResult.getErrorDetails());
                 throw new RuntimeException(compilationResult.getErrorDetails());
             }
@@ -174,12 +198,16 @@ public class RemoteJsInvokeService extends AbstractJsInvokeService {
                 .setInvokeRequest(jsRequestBuilder.build())
                 .build();
 
+
         ListenableFuture<JsInvokeProtos.RemoteJsResponse> future = kafkaTemplate.post(scriptId.toString(), jsRequestWrapper);
+        kafkaPushedMsgs.incrementAndGet();
         return Futures.transform(future, response -> {
             JsInvokeProtos.JsInvokeResponse invokeResult = response.getInvokeResponse();
             if (invokeResult.getSuccess()) {
+                kafkaInvokeMsgs.incrementAndGet();
                 return invokeResult.getResult();
             } else {
+                kafkaFailedMsgs.incrementAndGet();
                 log.debug("[{}] Failed to compile script due to [{}]: {}", scriptId, invokeResult.getErrorCode().name(), invokeResult.getErrorDetails());
                 throw new RuntimeException(invokeResult.getErrorDetails());
             }

@@ -19,6 +19,7 @@ import com.google.common.hash.HashFunction;
 import com.google.common.hash.Hashing;
 
 import java.nio.charset.Charset;
+import java.util.Random;
 
 
 /**

@@ -246,6 +246,7 @@ actors:
   statistics:
     # Enable/disable actor statistics
     enabled: "${ACTORS_STATISTICS_ENABLED:true}"
+    js_print_interval_ms: "${ACTORS_JS_STATISTICS_PRINT_INTERVAL_MS:10000}"
     persist_frequency: "${ACTORS_STATISTICS_PERSIST_FREQUENCY:3600000}"
   queue:
     # Enable/disable persistence of un-processed messages to the queue
@@ -467,6 +468,9 @@ js:
     response_auto_commit_interval: "${REMOTE_JS_RESPONSE_AUTO_COMMIT_INTERVAL_MS:100}"
     # Maximum allowed JavaScript execution errors before JavaScript will be blacklisted
     max_errors: "${REMOTE_JS_SANDBOX_MAX_ERRORS:3}"
+    stats:
+      enabled: "${TB_JS_REMOTE_STATS_ENABLED:false}"
+      print_interval_ms: "${TB_JS_REMOTE_STATS_PRINT_INTERVAL_MS:10000}"
 
 transport:
   type: "${TRANSPORT_TYPE:local}" # local or remote

+/**
+ * Copyright © 2016-2019 The Thingsboard Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.thingsboard.server.service.cluster.routing;
+
+import com.datastax.driver.core.utils.UUIDs;
+import lombok.extern.slf4j.Slf4j;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.runners.MockitoJUnitRunner;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.test.util.ReflectionTestUtils;
+import org.thingsboard.server.common.data.Device;
+import org.thingsboard.server.common.data.UUIDConverter;
+import org.thingsboard.server.common.data.id.DeviceId;
+import org.thingsboard.server.common.msg.cluster.ServerAddress;
+import org.thingsboard.server.common.msg.cluster.ServerType;
+import org.thingsboard.server.service.cluster.discovery.DiscoveryService;
+import org.thingsboard.server.service.cluster.discovery.ServerInstance;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Paths;
+import java.util.ArrayList;
+import java.util.Comparator;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.UUID;
+import java.util.stream.Collectors;
+
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+@Slf4j
+@RunWith(MockitoJUnitRunner.class)
+public class ConsistentClusterRoutingServiceTest {
+
+    private ConsistentClusterRoutingService clusterRoutingService;
+
+    private DiscoveryService discoveryService;
+
+    private String hashFunctionName = "murmur3_128";
+    private Integer virtualNodesSize = 1024*64;
+    private ServerAddress currentServer = new ServerAddress(" 100.96.1.0", 9001, ServerType.CORE);
+
+
+    @Before
+    public void setup() throws Exception {
+        discoveryService = mock(DiscoveryService.class);
+        clusterRoutingService = new ConsistentClusterRoutingService();
+        ReflectionTestUtils.setField(clusterRoutingService, "discoveryService", discoveryService);
+        ReflectionTestUtils.setField(clusterRoutingService, "hashFunctionName", hashFunctionName);
+        ReflectionTestUtils.setField(clusterRoutingService, "virtualNodesSize", virtualNodesSize);
+        when(discoveryService.getCurrentServer()).thenReturn(new ServerInstance(currentServer));
+        List<ServerInstance> otherServers = new ArrayList<>();
+        for (int i = 1; i < 30; i++) {
+            otherServers.add(new ServerInstance(new ServerAddress(" 100.96." + i*2 + "." + i, 9001, ServerType.CORE)));
+        }
+        when(discoveryService.getOtherServers()).thenReturn(otherServers);
+        clusterRoutingService.init();
+    }
+
+    @Test
+    public void testDispersionOnMillionDevices() {
+        List<DeviceId> devices = new ArrayList<>();
+        for (int i = 0; i < 1000000; i++) {
+            devices.add(new DeviceId(UUIDs.timeBased()));
+        }
+
+        testDevicesDispersion(devices);
+    }
+
+    @Test
+    public void testDispersionOnDevicesFromFile() throws IOException {
+        List<String> deviceIdsStrList = Files.readAllLines(Paths.get("/home/ashvayka/Downloads/deviceIds.out"));
+        List<DeviceId> devices = deviceIdsStrList.stream().map(String::trim).filter(s -> !s.isEmpty()).map(UUIDConverter::fromString).map(DeviceId::new).collect(Collectors.toList());
+        System.out.println("Devices: " + devices.size());
+        testDevicesDispersion(devices);
+        testDevicesDispersion(devices);
+        testDevicesDispersion(devices);
+        testDevicesDispersion(devices);
+        testDevicesDispersion(devices);
+
+    }
+
+    private void testDevicesDispersion(List<DeviceId> devices) {
+        long start = System.currentTimeMillis();
+        Map<ServerAddress, Integer> map = new HashMap<>();
+        for (DeviceId deviceId : devices) {
+            ServerAddress address = clusterRoutingService.resolveById(deviceId).orElse(currentServer);
+            map.put(address, map.getOrDefault(address, 0) + 1);
+        }
+
+        List<Map.Entry<ServerAddress, Integer>> data = map.entrySet().stream().sorted(Comparator.comparingInt(Map.Entry::getValue)).collect(Collectors.toList());
+        long end = System.currentTimeMillis();
+        System.out.println("Size: " + virtualNodesSize + " Time: " + (end - start) + " Diff: " + (data.get(data.size() - 1).getValue() - data.get(0).getValue()));
+
+        for (Map.Entry<ServerAddress, Integer> entry : data) {
+//            System.out.println(entry.getKey().getHost() + ": " + entry.getValue());
+        }
+
+    }
+
+}

@@ -123,6 +123,12 @@ public interface TbContext {
 
     ScriptEngine createJsScriptEngine(String script, String... argNames);
 
+    void logJsEvalRequest();
+
+    void logJsEvalResponse();
+
+    void logJsEvalFailure();
+
     String getNodeId();
 
     RuleChainTransactionService getRuleChainTransactionService();

@@ -65,8 +65,10 @@ public class TbClearAlarmNode extends TbAbstractAlarmNode<TbClearAlarmNodeConfig
     }
 
     private ListenableFuture<AlarmResult> clearAlarm(TbContext ctx, TbMsg msg, Alarm alarm) {
+        ctx.logJsEvalRequest();
         ListenableFuture<JsonNode> asyncDetails = buildAlarmDetails(ctx, msg, alarm.getDetails());
         return Futures.transformAsync(asyncDetails, details -> {
+            ctx.logJsEvalResponse();
             ListenableFuture<Boolean> clearFuture = ctx.getAlarmService().clearAlarm(ctx.getTenantId(), alarm.getId(), details, System.currentTimeMillis());
             return Futures.transformAsync(clearFuture, cleared -> {
                 ListenableFuture<Alarm> savedAlarmFuture = ctx.getAlarmService().findAlarmByIdAsync(ctx.getTenantId(), alarm.getId());

@@ -42,10 +42,10 @@ import java.io.IOException;
         nodeDescription = "Create or Update Alarm",
         nodeDetails =
                 "Details - JS function that creates JSON object based on incoming message. This object will be added into Alarm.details field.\n" +
-                "Node output:\n" +
-                "If alarm was not created, original message is returned. Otherwise new Message returned with type 'ALARM', Alarm object in 'msg' property and 'matadata' will contains one of those properties 'isNewAlarm/isExistingAlarm'. " +
-                "Message payload can be accessed via <code>msg</code> property. For example <code>'temperature = ' + msg.temperature ;</code>. " +
-                "Message metadata can be accessed via <code>metadata</code> property. For example <code>'name = ' + metadata.customerName;</code>.",
+                        "Node output:\n" +
+                        "If alarm was not created, original message is returned. Otherwise new Message returned with type 'ALARM', Alarm object in 'msg' property and 'matadata' will contains one of those properties 'isNewAlarm/isExistingAlarm'. " +
+                        "Message payload can be accessed via <code>msg</code> property. For example <code>'temperature = ' + msg.temperature ;</code>. " +
+                        "Message metadata can be accessed via <code>metadata</code> property. For example <code>'name = ' + metadata.customerName;</code>.",
         uiResources = {"static/rulenode/rulenode-core-config.js"},
         configDirective = "tbActionNodeCreateAlarmConfig",
         icon = "notifications_active"
@@ -103,11 +103,15 @@ public class TbCreateAlarmNode extends TbAbstractAlarmNode<TbCreateAlarmNodeConf
 
     private ListenableFuture<AlarmResult> createNewAlarm(TbContext ctx, TbMsg msg, Alarm msgAlarm) {
         ListenableFuture<Alarm> asyncAlarm;
-        if (msgAlarm != null ) {
+        if (msgAlarm != null) {
             asyncAlarm = Futures.immediateCheckedFuture(msgAlarm);
         } else {
+            ctx.logJsEvalRequest();
             asyncAlarm = Futures.transform(buildAlarmDetails(ctx, msg, null),
-                    details -> buildAlarm(msg, details, ctx.getTenantId()));
+                    details -> {
+                        ctx.logJsEvalResponse();
+                        return buildAlarm(msg, details, ctx.getTenantId());
+                    });
         }
         ListenableFuture<Alarm> asyncCreated = Futures.transform(asyncAlarm,
                 alarm -> ctx.getAlarmService().createOrUpdateAlarm(alarm), ctx.getDbCallbackExecutor());
@@ -115,7 +119,9 @@ public class TbCreateAlarmNode extends TbAbstractAlarmNode<TbCreateAlarmNodeConf
     }
 
     private ListenableFuture<AlarmResult> updateAlarm(TbContext ctx, TbMsg msg, Alarm existingAlarm, Alarm msgAlarm) {
+        ctx.logJsEvalRequest();
         ListenableFuture<Alarm> asyncUpdated = Futures.transform(buildAlarmDetails(ctx, msg, existingAlarm.getDetails()), (Function<JsonNode, Alarm>) details -> {
+            ctx.logJsEvalResponse();
             if (msgAlarm != null) {
                 existingAlarm.setSeverity(msgAlarm.getSeverity());
                 existingAlarm.setPropagate(msgAlarm.isPropagate());

@@ -53,12 +53,17 @@ public class TbLogNode implements TbNode {
     @Override
     public void onMsg(TbContext ctx, TbMsg msg) {
         ListeningExecutor jsExecutor = ctx.getJsExecutor();
+        ctx.logJsEvalRequest();
         withCallback(jsExecutor.executeAsync(() -> jsEngine.executeToString(msg)),
                 toString -> {
+                    ctx.logJsEvalResponse();
                     log.info(toString);
                     ctx.tellNext(msg, SUCCESS);
                 },
-                t -> ctx.tellFailure(msg, t));
+                t -> {
+                    ctx.logJsEvalResponse();
+                    ctx.tellFailure(msg, t);
+                });
     }
 
     @Override

@@ -129,7 +129,9 @@ public class TbMsgGeneratorNode implements TbNode {
                 prevMsg = ctx.newMsg("", originatorId, new TbMsgMetaData(), "{}");
             }
             if (initialized) {
+                ctx.logJsEvalRequest();
                 TbMsg generated = jsEngine.executeGenerate(prevMsg);
+                ctx.logJsEvalResponse();
                 prevMsg = ctx.newMsg(generated.getType(), originatorId, generated.getMetaData(), generated.getData());
             }
             return prevMsg;

@@ -52,9 +52,16 @@ public class TbJsFilterNode implements TbNode {
     @Override
     public void onMsg(TbContext ctx, TbMsg msg) {
         ListeningExecutor jsExecutor = ctx.getJsExecutor();
+        ctx.logJsEvalRequest();
         withCallback(jsExecutor.executeAsync(() -> jsEngine.executeFilter(msg)),
-                filterResult -> ctx.tellNext(msg, filterResult ? "True" : "False"),
-                t -> ctx.tellFailure(msg, t));
+                filterResult -> {
+                    ctx.logJsEvalResponse();
+                    ctx.tellNext(msg, filterResult ? "True" : "False");
+                },
+                t -> {
+                    ctx.tellFailure(msg, t);
+                    ctx.logJsEvalFailure();
+                }, ctx.getDbCallbackExecutor());
     }
 
     @Override

@@ -54,9 +54,16 @@ public class TbJsSwitchNode implements TbNode {
     @Override
     public void onMsg(TbContext ctx, TbMsg msg) {
         ListeningExecutor jsExecutor = ctx.getJsExecutor();
+        ctx.logJsEvalRequest();
         withCallback(jsExecutor.executeAsync(() -> jsEngine.executeSwitch(msg)),
-                result -> processSwitch(ctx, msg, result),
-                t -> ctx.tellFailure(msg, t));
+                result -> {
+                    ctx.logJsEvalResponse();
+                    processSwitch(ctx, msg, result);
+                },
+                t -> {
+                    ctx.logJsEvalFailure();
+                    ctx.tellFailure(msg, t);
+                }, ctx.getDbCallbackExecutor());
     }
 
     private void processSwitch(TbContext ctx, TbMsg msg, Set<String> nextRelations) {

@@ -44,14 +44,21 @@ public abstract class TbAbstractTransformNode implements TbNode {
     @Override
     public void onMsg(TbContext ctx, TbMsg msg) {
         withCallback(transform(ctx, msg),
-                m -> {
-                    if (m != null) {
-                        ctx.tellNext(m, SUCCESS);
-                    } else {
-                        ctx.tellNext(msg, FAILURE);
-                    }
-                },
-                t -> ctx.tellFailure(msg, t));
+                m -> transformSuccess(ctx, msg, m),
+                t -> transformFailure(ctx, msg, t),
+                ctx.getDbCallbackExecutor());
+    }
+
+    protected void transformFailure(TbContext ctx, TbMsg msg, Throwable t) {
+        ctx.tellFailure(msg, t);
+    }
+
+    protected void transformSuccess(TbContext ctx, TbMsg msg, TbMsg m) {
+        if (m != null) {
+            ctx.tellNext(m, SUCCESS);
+        } else {
+            ctx.tellNext(msg, FAILURE);
+        }
     }
 
     protected abstract ListenableFuture<TbMsg> transform(TbContext ctx, TbMsg msg);

@@ -21,6 +21,9 @@ import org.thingsboard.rule.engine.api.*;
 import org.thingsboard.server.common.data.plugin.ComponentType;
 import org.thingsboard.server.common.msg.TbMsg;
 
+import static org.thingsboard.rule.engine.api.TbRelationTypes.FAILURE;
+import static org.thingsboard.rule.engine.api.TbRelationTypes.SUCCESS;
+
 @RuleNode(
         type = ComponentType.TRANSFORMATION,
         name = "script",
@@ -49,10 +52,23 @@ public class TbTransformMsgNode extends TbAbstractTransformNode {
 
     @Override
     protected ListenableFuture<TbMsg> transform(TbContext ctx, TbMsg msg) {
+        ctx.logJsEvalRequest();
         return ctx.getJsExecutor().executeAsync(() -> jsEngine.executeUpdate(msg));
     }
 
     @Override
+    protected void transformSuccess(TbContext ctx, TbMsg msg, TbMsg m) {
+        ctx.logJsEvalResponse();
+        super.transformSuccess(ctx, msg, m);
+    }
+
+    @Override
+    protected void transformFailure(TbContext ctx, TbMsg msg, Throwable t) {
+        ctx.logJsEvalFailure();
+        super.transformFailure(ctx, msg, t);
+    }
+
+    @Override
     public void destroy() {
         if (jsEngine != null) {
             jsEngine.destroy();