diff --git a/tests/script/sh/bit_and.c b/tests/script/sh/bit_and.c
index 2cf2157e1c..84485d396b 100644
--- a/tests/script/sh/bit_and.c
+++ b/tests/script/sh/bit_and.c
@@ -55,7 +55,7 @@ DLL_EXPORT int32_t bit_and(SUdfDataBlock* block, SUdfColumn* resultCol) {
   }
 
   resultData->numOfRows = block->numOfRows;
-  udfTrace("block:%p, processing completed, rows:%d, cols:%d,", block, block->numOfRows, block->numOfCols);
+  udfTrace("block:%p, processing completed", block);
 
   return TSDB_CODE_SUCCESS;
 }
diff --git a/tests/script/sh/l2norm.c b/tests/script/sh/l2norm.c
index 0b7f5bf7f6..865d9ee9a5 100644
--- a/tests/script/sh/l2norm.c
+++ b/tests/script/sh/l2norm.c
@@ -1,32 +1,26 @@
-#include <string.h>
-#include <stdlib.h>
-#include <stdio.h>
 #include <math.h>
-
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 #include "taosudf.h"
 
-DLL_EXPORT int32_t l2norm_init() {
-  return 0;
-}
+DLL_EXPORT int32_t l2norm_init() { return 0; }
 
-DLL_EXPORT int32_t l2norm_destroy() {
-  return 0;
-}
+DLL_EXPORT int32_t l2norm_destroy() { return 0; }
 
-DLL_EXPORT int32_t l2norm_start(SUdfInterBuf *buf) {
+DLL_EXPORT int32_t l2norm_start(SUdfInterBuf* buf) {
   *(int64_t*)(buf->buf) = 0;
   buf->bufLen = sizeof(double);
   buf->numOfResult = 1;
   return 0;
 }
 
-DLL_EXPORT int32_t l2norm(SUdfDataBlock* block, SUdfInterBuf *interBuf, SUdfInterBuf *newInterBuf) {
+DLL_EXPORT int32_t l2norm(SUdfDataBlock* block, SUdfInterBuf* interBuf, SUdfInterBuf* newInterBuf) {
   double sumSquares = *(double*)interBuf->buf;
   int8_t numNotNull = 0;
   for (int32_t i = 0; i < block->numOfCols; ++i) {
     SUdfColumn* col = block->udfCols[i];
-    if (!(col->colMeta.type == TSDB_DATA_TYPE_INT || 
-          col->colMeta.type == TSDB_DATA_TYPE_DOUBLE)) {
+    if (!(col->colMeta.type == TSDB_DATA_TYPE_INT || col->colMeta.type == TSDB_DATA_TYPE_DOUBLE)) {
       return TSDB_CODE_UDF_INVALID_INPUT;
     }
   }
@@ -38,18 +32,18 @@ DLL_EXPORT int32_t l2norm(SUdfDataBlock* block, SUdfInterBuf *interBuf, SUdfInte
       }
       switch (col->colMeta.type) {
         case TSDB_DATA_TYPE_INT: {
-          char* cell = udfColDataGetData(col, j);
+          char*   cell = udfColDataGetData(col, j);
           int32_t num = *(int32_t*)cell;
           sumSquares += (double)num * num;
           break;
         }
         case TSDB_DATA_TYPE_DOUBLE: {
-          char* cell = udfColDataGetData(col, j);
+          char*  cell = udfColDataGetData(col, j);
           double num = *(double*)cell;
           sumSquares += num * num;
           break;
         }
-        default: 
+        default:
           break;
       }
       ++numNotNull;
@@ -62,7 +56,7 @@ DLL_EXPORT int32_t l2norm(SUdfDataBlock* block, SUdfInterBuf *interBuf, SUdfInte
   return 0;
 }
 
-DLL_EXPORT int32_t l2norm_finish(SUdfInterBuf* buf, SUdfInterBuf *resultData) {
+DLL_EXPORT int32_t l2norm_finish(SUdfInterBuf* buf, SUdfInterBuf* resultData) {
   double sumSquares = *(double*)(buf->buf);
   *(double*)(resultData->buf) = sqrt(sumSquares);
   resultData->bufLen = sizeof(double);
diff --git a/tests/script/sh/max_vol.c b/tests/script/sh/max_vol.c
index 4f9ecd33a7..0a57a26d1c 100644
--- a/tests/script/sh/max_vol.c
+++ b/tests/script/sh/max_vol.c
@@ -1,101 +1,90 @@
-#include <string.h>
-#include <stdlib.h>
-#include <stdio.h>
 #include <math.h>
-
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 #include "taosudf.h"
 
-#define STR_MAX_LEN 256 // inter buffer length
+#define STR_MAX_LEN 256  // inter buffer length
 
 // init
-DLL_EXPORT int32_t max_vol_init()
-{
-    return 0;
-}
+DLL_EXPORT int32_t max_vol_init() { return 0; }
 
 // destory
-DLL_EXPORT int32_t max_vol_destroy()
-{
-    return 0;
-}
+DLL_EXPORT int32_t max_vol_destroy() { return 0; }
 
-// start 
-DLL_EXPORT int32_t max_vol_start(SUdfInterBuf *buf)
-{
-    memset(buf->buf, 0, sizeof(float) + STR_MAX_LEN);
-    // set init value
-    *((float*)buf->buf) = -10000000;
-    buf->bufLen = sizeof(float)  + STR_MAX_LEN;
-    buf->numOfResult = 0;
-    return 0;
+// start
+DLL_EXPORT int32_t max_vol_start(SUdfInterBuf *buf) {
+  memset(buf->buf, 0, sizeof(float) + STR_MAX_LEN);
+  // set init value
+  *((float *)buf->buf) = -10000000;
+  buf->bufLen = sizeof(float) + STR_MAX_LEN;
+  buf->numOfResult = 0;
+  return 0;
 }
 
 DLL_EXPORT int32_t max_vol(SUdfDataBlock *block, SUdfInterBuf *interBuf, SUdfInterBuf *newInterBuf) {
-    float maxValue = *(float *)interBuf->buf;
-    char strBuff[STR_MAX_LEN] = "inter1buf";
-    
-    if (block->numOfCols < 2)
-    {
+  float maxValue = *(float *)interBuf->buf;
+  char  strBuff[STR_MAX_LEN] = "inter1buf";
+
+  if (block->numOfCols < 2) {
+    return TSDB_CODE_UDF_INVALID_INPUT;
+  }
+
+  // check data type
+  for (int32_t i = 0; i < block->numOfCols; ++i) {
+    SUdfColumn *col = block->udfCols[i];
+    if (i == block->numOfCols - 1) {
+      // last column is device id , must varchar
+      if (col->colMeta.type != TSDB_DATA_TYPE_VARCHAR) {
         return TSDB_CODE_UDF_INVALID_INPUT;
+      }
+    } else {
+      if (col->colMeta.type != TSDB_DATA_TYPE_FLOAT) {
+        return TSDB_CODE_UDF_INVALID_INPUT;
+      }
     }
+  }
 
-    // check data type
-    for (int32_t i = 0; i < block->numOfCols; ++i)
-    {
-        SUdfColumn *col = block->udfCols[i];
-        if( i == block->numOfCols - 1) {
-          // last column is device id , must varchar
-          if (col->colMeta.type != TSDB_DATA_TYPE_VARCHAR ) {
-            return TSDB_CODE_UDF_INVALID_INPUT;
-          }
-        } else {
-          if (col->colMeta.type != TSDB_DATA_TYPE_FLOAT) {
-            return TSDB_CODE_UDF_INVALID_INPUT;
-          }
-        }
+  // calc max voltage
+  SUdfColumn *lastCol = block->udfCols[block->numOfCols - 1];
+  for (int32_t i = 0; i < (block->numOfCols - 1); ++i) {
+    for (int32_t j = 0; j < block->numOfRows; ++j) {
+      SUdfColumn *col = block->udfCols[i];
+      if (udfColDataIsNull(col, j)) {
+        continue;
+      }
+      char *data = udfColDataGetData(col, j);
+      float voltage = *(float *)data;
+      if (voltage > maxValue) {
+        maxValue = voltage;
+        char *valData = udfColDataGetData(lastCol, j);
+        // get device id
+        char *deviceId = valData + sizeof(uint16_t);
+        sprintf(strBuff, "%s_(%d,%d)_%f", deviceId, j, i, maxValue);
+      }
     }
+  }
 
-    // calc max voltage
-    SUdfColumn *lastCol = block->udfCols[block->numOfCols - 1];
-    for (int32_t i = 0; i < (block->numOfCols - 1); ++i) {
-        for (int32_t j = 0; j < block->numOfRows; ++j) {
-          SUdfColumn *col = block->udfCols[i];
-          if (udfColDataIsNull(col, j)) {
-            continue;
-          }
-          char *data = udfColDataGetData(col, j);
-          float voltage = *(float *)data;
-          if (voltage > maxValue) {
-            maxValue = voltage;
-            char *valData = udfColDataGetData(lastCol, j);
-            // get device id
-            char *deviceId = valData + sizeof(uint16_t);
-            sprintf(strBuff, "%s_(%d,%d)_%f", deviceId, j, i, maxValue);
-          }
-        }
-    }
-
-    *(float*)newInterBuf->buf = maxValue;
-    strcpy(newInterBuf->buf + sizeof(float), strBuff);
-    newInterBuf->bufLen = sizeof(float) + strlen(strBuff)+1;
-    newInterBuf->numOfResult = 1;
-    return 0;
+  *(float *)newInterBuf->buf = maxValue;
+  strcpy(newInterBuf->buf + sizeof(float), strBuff);
+  newInterBuf->bufLen = sizeof(float) + strlen(strBuff) + 1;
+  newInterBuf->numOfResult = 1;
+  return 0;
 }
 
-DLL_EXPORT int32_t max_vol_finish(SUdfInterBuf *buf, SUdfInterBuf *resultData)
-{
-    char * str = buf->buf + sizeof(float);
-    // copy to des
-    char * des = resultData->buf + sizeof(uint16_t);
-    strcpy(des, str);
+DLL_EXPORT int32_t max_vol_finish(SUdfInterBuf *buf, SUdfInterBuf *resultData) {
+  char *str = buf->buf + sizeof(float);
+  // copy to des
+  char *des = resultData->buf + sizeof(uint16_t);
+  strcpy(des, str);
 
-    // set binary type len
-    uint16_t  len = strlen(str);
-    *((uint16_t*)resultData->buf) = len;
+  // set binary type len
+  uint16_t len = strlen(str);
+  *((uint16_t *)resultData->buf) = len;
 
-    // set buf len
-    resultData->bufLen = len + sizeof(uint16_t);
-    // set row count
-    resultData->numOfResult = 1;
-    return 0;
+  // set buf len
+  resultData->bufLen = len + sizeof(uint16_t);
+  // set row count
+  resultData->numOfResult = 1;
+  return 0;
 }