Merge pull request #908 from taosdata/feature/slguan

remove some warnings

src/client/src/sql.c

@@ -22,6 +22,8 @@
 ** The following is the concatenation of all %include directives from the
 ** input grammar file:
 */
+#pragma GCC diagnostic ignored "-Wunused-variable"
+
 #include <stdio.h>
 /************ Begin %include sections from the grammar ************************/
 

src/client/src/tscServer.c

@@ -211,7 +211,6 @@ void tscGetConnToMgmt(SSqlObj *pSql, uint8_t *pCode) {
 }
 
 void tscGetConnToVnode(SSqlObj *pSql, uint8_t *pCode) {
-  char        ipstr[40] = {0};
   SVPeerDesc *pVPeersDesc = NULL;
   static int  vidIndex = 0;
   STscObj *   pTscObj = pSql->pTscObj;
@@ -244,6 +243,7 @@ void tscGetConnToVnode(SSqlObj *pSql, uint8_t *pCode) {
   while (pSql->retry < pSql->maxRetry) {
     (pSql->retry)++;
 #ifdef CLUSTER
+    char ipstr[40] = {0};
     if (pVPeersDesc[pSql->index].ip == 0) {
       (pSql->index) = (pSql->index + 1) % TSDB_VNODES_SUPPORT;
       continue;

src/modules/monitor/src/monitorSystem.c

@@ -389,21 +389,29 @@ void monitorSaveAcctLog(char *acctId, int64_t currentPointsPerSecond, int64_t ma
   char sql[1024] = {0};
   sprintf(sql,
           "insert into %s.acct_%s using %s.acct tags('%s') values(now"
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
-	  ", %" PRId64, "%" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
+	      ", %" PRId64 ", %" PRId64
           ", %d)",
-          tsMonitorDbName, acctId, tsMonitorDbName, acctId, currentPointsPerSecond, maxPointsPerSecond, totalTimeSeries,
-          maxTimeSeries, totalStorage, maxStorage, totalQueryTime, maxQueryTime, totalInbound, maxInbound,
-          totalOutbound, maxOutbound, totalDbs, maxDbs, totalUsers, maxUsers, totalStreams, maxStreams, totalConns,
-          maxConns, accessState);
+          tsMonitorDbName, acctId, tsMonitorDbName, acctId,
+          currentPointsPerSecond, maxPointsPerSecond,
+          totalTimeSeries, maxTimeSeries,
+          totalStorage, maxStorage,
+          totalQueryTime, maxQueryTime,
+          totalInbound, maxInbound,
+          totalOutbound, maxOutbound,
+          totalDbs, maxDbs,
+          totalUsers, maxUsers,
+          totalStreams, maxStreams,
+          totalConns, maxConns,
+          accessState);
 
   monitorTrace("monitor:%p, save account info, sql %s", monitor->conn, sql);
   taos_query_a(monitor->conn, sql, dnodeMontiorInsertAcctCallback, "account");
@@ -418,7 +426,7 @@ void monitorSaveLog(int level, const char *const format, ...) {
     return;
   }
 
-  int len = snprintf(sql, (size_t)max_length, "import into %s.log values(%\" PRId64 \", %d,'", tsMonitorDbName,
+  int len = snprintf(sql, (size_t)max_length, "import into %s.log values(%" PRId64 ", %d,'", tsMonitorDbName,
                      taosGetTimestampUs(), level);
 
   va_start(argpointer, format);