Changeset 852 for trunk/Modules/NetworkConfigRouterOS/Generators/NetwatchImport.php

Timestamp:

Oct 25, 2017, 10:55:05 PM (7 years ago)

Author:

chronos

Message:

• Modified: Use better transaction mechanism for netwatch import and latency measurement.
• Fixed: Query duration mesurement using outdated microtime result.

File:

• trunk/Modules/NetworkConfigRouterOS/Generators/NetwatchImport.php (modified) (4 diffs)

trunk/Modules/NetworkConfigRouterOS/Generators/NetwatchImport.php

@@ -42 +42 @@
     }
 
-    $this->Database->query('BEGIN');
-
+    $Queries = array();
     foreach($Interfaces as $Index => $Interface)
     {
       // Update last online time if still online
       if($Interface['NewOnline'])
-        $DbResult = $this->Database->update('NetworkInterface', '`Id` = "'.$Interface['Id'].'"',
+        $Queries[] = $this->Database->GetUpdate('NetworkInterface', '`Id` = "'.$Interface['Id'].'"',
           array('LastOnline' => TimeToMysqlDateTime($StartTime)));
 
@@ -54 +53 @@
       {
         // Online state changed
-        $DbResult = $this->Database->query('INSERT INTO `NetworkInterfaceUpDown` (`Interface`,
+        $Queries[] = 'INSERT INTO `NetworkInterfaceUpDown` (`Interface`,
          `State`, `Time`, `Duration`) VALUES ('.$Interface['Id'].', '.$Interface['NewOnline'].', "'.
-          TimeToMysqlDateTime($StartTime).'", NULL)');
+          TimeToMysqlDateTime($StartTime).'", NULL)';
         // Update previous record duration in UpDown table
-        $this->Database->query('UPDATE `NetworkInterfaceUpDown` AS `TM` SET `Duration` = TIMESTAMPDIFF(SECOND, '.
+        $Queries[] = 'UPDATE `NetworkInterfaceUpDown` AS `TM` SET `Duration` = TIMESTAMPDIFF(SECOND, '.
           '`TM`.`Time`, (SELECT `Time` FROM (SELECT * FROM `NetworkInterfaceUpDown`) AS `TA` WHERE (`TA`.`Time` > `TM`.`Time`) '.
           'AND (`TA`.`Interface`=`TM`.`Interface`) ORDER BY `TA`.`Time` ASC LIMIT 1)) '.
-          'WHERE (`TM`.`Duration` IS NULL) AND (`TM`.`Interface` ='.$Interface['Id'].')');
-        $this->Database->update('NetworkInterface', '`Id` = "'.$Interface['Id'].'"',
+          'WHERE (`TM`.`Duration` IS NULL) AND (`TM`.`Interface` ='.$Interface['Id'].')';
+        $Queries[] = $this->Database->GetUpdate('NetworkInterface', '`Id` = "'.$Interface['Id'].'"',
           array('Online' => $Interface['NewOnline']));
       }
     }
+    $this->Database->Transaction($Queries);
 
     // Set offline all interfaces which were not updated as online
@@ -81 +81 @@
     }
 
+    $Queries = array();
     // Update device online state
     $DbResult = $this->Database->select('NetworkInterface', '`Device`, SUM(`Online`) AS `SumOnline`', '`Online` = 1 GROUP BY `Device`');
@@ -86 +87 @@
     {
       if($Device['SumOnline'] > 0)
-        $this->Database->update('NetworkDevice', 'Id='.$Device['Device'], array('LastOnline' => TimeToMysqlDateTime($StartTime), 'Online' => 1));
+        $Queries[] = $this->Database->GetUpdate('NetworkDevice', 'Id='.$Device['Device'], array('LastOnline' => TimeToMysqlDateTime($StartTime), 'Online' => 1));
     }
-    $this->Database->update('NetworkDevice', '`LastOnline` < "'.TimeToMysqlDateTime($StartTime).'"', array('Online' => 0));
-
-    $this->Database->query('COMMIT');
+    $Queries[] = $this->Database->GetUpdate('NetworkDevice', '`LastOnline` < "'.TimeToMysqlDateTime($StartTime).'"', array('Online' => 0));
+    $this->Database->Transaction($Queries);
   }