refs #5563 Fist showValue implementation

lib/model-loader.js

@@ -1,12 +1,14 @@
 const path = require('path');
 const {loadConfig, toUpperCamelCase} = require('./util');
+const MultiMap = require('./multi-map');
 
 module.exports = class ModelLoader {
   init(conf) {
     const configDir = path.join(__dirname, '..');
     const logsConf = this.logsConf = loadConfig(configDir, 'logs');
-    const schemaMap = new Map();
+    const schemaMap = new MultiMap();
     const logMap = new Map();
+    const showTables = new MultiMap();
   
     for (const logName in logsConf.logs) {
       const logConf = logsConf.logs[logName];
@@ -40,19 +42,13 @@ module.exports = class ModelLoader {
         tableConf = {name: tableConf};
       const table = parseTable(tableConf.name, logInfo.schema);
   
-      let tableMap = schemaMap.get(table.schema);
-      if (!tableMap) {
-        tableMap = new Map();
-        schemaMap.set(table.schema, tableMap);
-      }
-  
-      let tableInfo = tableMap.get(table.name);
+      let tableInfo = schemaMap.get(table.schema, table.name);
       if (!tableInfo) {
         tableInfo = {
           conf: tableConf,
           log: logInfo
         };
-        tableMap.set(table.name, tableInfo);
+        schemaMap.set(table.schema, table.name, tableInfo);
       }
   
       let modelName = tableConf.modelName;
@@ -81,14 +77,13 @@ module.exports = class ModelLoader {
       return tableInfo;
     }
   
-    return {schemaMap, logMap};
+    return {schemaMap, logMap, showTables};
   }
 
-  async loadSchema(schemaMap, db) {
+  async loadSchema(db, schemaMap, showTables) {
     const {logsConf} = this;
   
-    for (const [schema, tableMap] of schemaMap)
-    for (const [table, tableInfo] of tableMap) {
+    for (const [schema, table, tableInfo] of schemaMap) {
       const tableConf = tableInfo.conf;
   
       // Fetch columns & types
@@ -156,14 +151,10 @@ module.exports = class ModelLoader {
 
     // Fetch relation to main table
 
-    for (const [schema, tableMap] of schemaMap)
-    for (const [table, tableInfo] of tableMap) {
-  
+    for (const [schema, table, tableInfo] of schemaMap) {
       if (!tableInfo.conf.relation && !tableInfo.isMain) {
         const mainTable = tableInfo.log.mainTable;
-        const mainTableInfo = schemaMap
-          .get(mainTable.schema)
-          .get(mainTable.name);
+        const mainInfo = schemaMap.get(mainTable.schema, mainTable.name);
   
         const [mainRelations] = await db.query(
           `SELECT COLUMN_NAME relation
@@ -178,7 +169,7 @@ module.exports = class ModelLoader {
             schema,
             mainTable.name,
             mainTable.schema,
-            mainTableInfo.idName
+            mainInfo.idName
           ]
         );
   
@@ -192,69 +183,108 @@ module.exports = class ModelLoader {
       }
     }
   
-    // Fetch relations and show values of related tables
-    // TODO: #5563 Not used yet 
+    // Fetch relations with other tables
+    // TODO: #5563 Fetch relations and show values in fronted
   
-    const relatedList = [];
-    const relatedMap = new Map();
+    showTables.clear();
 
-    for (const [schema, tableMap] of schemaMap)
-    for (const [table, tableInfo] of tableMap) {
+    for (const [schema, table, tableInfo] of schemaMap) {
       const [relations] = await db.query(
         `SELECT
             COLUMN_NAME \`col\`,
             REFERENCED_TABLE_SCHEMA \`schema\`,
-            REFERENCED_TABLE_NAME \`table\`,
-            REFERENCED_COLUMN_NAME \`column\`
+            REFERENCED_TABLE_NAME \`table\`
           FROM information_schema.KEY_COLUMN_USAGE
           WHERE TABLE_NAME = ?
             AND TABLE_SCHEMA = ?
+            AND COLUMN_NAME IN (?)
             AND REFERENCED_TABLE_NAME IS NOT NULL`,
-        [table, schema]
+        [
+          table,
+          schema,
+          Array.from(tableInfo.columns.keys())
+        ]
       );
   
       tableInfo.relations = new Map();
-      for (const {col, schema, table, column} of relations) {
-        tableInfo.relations.set(col, {schema, table, column});
-        relatedList.push([table, schema]);
-  
-        let tables = relatedMap.get(schema);
-        if (!tables) relatedMap.set(schema, tables = new Set());
-        if (!tables.has(table)) {
-          tables.add(table);
-          relatedList.push([table, schema]);
-        }
+      for (const {col, schema, table} of relations) {
+        if (col == tableInfo.relation) continue;
+        tableInfo.relations.set(col, {schema, table});
+        showTables.setIfEmpty(schema, table, {});
       }
     }
 
+    const relatedList = Array.from(showTables.keys());
+
+    // Fetch primary key of related tables
+
+    const [res] = await db.query(
+      `SELECT
+          TABLE_SCHEMA \`schema\`,
+          TABLE_NAME \`table\`,
+          COLUMN_NAME \`idName\`,
+          COUNT(*) nPks
+        FROM information_schema.\`COLUMNS\`
+        WHERE (TABLE_SCHEMA, TABLE_NAME) IN (?)
+          AND COLUMN_KEY = 'PRI'
+        GROUP BY TABLE_NAME, TABLE_SCHEMA
+          HAVING nPks = 1`,
+      [relatedList]
+    );
+    for (const {schema, table, idName} of res)
+      showTables.get(schema, table).idName = idName;
+
+    // Fetch show field of related tables
+
     const showFields = logsConf.showFields;
     const [result] = await db.query(
       `SELECT
-          TABLE_NAME \`table\`,
           TABLE_SCHEMA \`schema\`,
+          TABLE_NAME \`table\`,
           COLUMN_NAME \`col\`
         FROM information_schema.\`COLUMNS\`
-        WHERE (TABLE_NAME, TABLE_SCHEMA) IN (?)
-          AND COLUMN_NAME IN (?)`,
+        WHERE (TABLE_SCHEMA, TABLE_NAME) IN (?)
+          AND COLUMN_NAME IN (?)
+          AND COLUMN_KEY <> 'PRI'`,
       [relatedList, showFields]
     );
 
-    const showTables = new Map();
-  
-    for (const {table, schema, col} of result) {
-      let tables = showTables.get(schema);
-      if (!tables) showTables.set(schema, tables = new Map())
-      const showField = tables.get(table);
-  
+    for (const {schema, table, col} of result) {
+      const tableInfo = showTables.get(schema, table);
       let save;
-      if (showField) {
+      if (tableInfo.showField) {
         const newIndex = showFields.indexOf(col);
-        const oldIndex = showFields.indexOf(showField);
+        const oldIndex = showFields.indexOf(tableInfo.showField);
         save = newIndex < oldIndex;
       } else
         save = true;
+      if (save) tableInfo.showField = col;
+    }
 
-      if (save) tables.set(table, col);
+    // Clean tables and relations without required information
+
+    for (const [schema, table] of relatedList) {
+      const tableInfo = showTables.get(schema, table);
+      const {idName, showField} = tableInfo;
+      if (!idName || !showField || idName == showField) {
+        showTables.delete(schema, table);
+        continue;
+      }
+
+      const sqlShowField = db.escapeId(showField);
+      const sqlIdName = db.escapeId(idName);
+      const sqlTable = `${db.escapeId(schema)}.${db.escapeId(table)}`;
+
+      tableInfo.selectStmt =
+        `SELECT ${sqlIdName} \`id\`, ${sqlShowField} \`val\`
+          FROM ${sqlTable}
+          WHERE ${sqlIdName} IN (?)`;
+    }
+
+    for (const tableInfo of schemaMap.values())
+    for (const [col, relation] of tableInfo.relations) {
+      if (!showTables.has(relation.schema, relation.table))
+        tableInfo.relations.delete(col);
     }
   }
 }

lib/multi-map.js

@@ -0,0 +1,58 @@
+module.exports = class MultiMap {
+  constructor() {
+    this.map = new Map();
+  }
+
+  set(key, subKey, value) {
+    let subKeys = this.map.get(key);
+    if (!subKeys) this.map.set(key, subKeys = new Map());
+    subKeys.set(subKey, value);
+  }
+
+  setIfEmpty(key, subKey, value) {
+    if (!this.has(key, subKey))
+      this.set(key, subKey, value);
+  }
+
+  get(key, subKey) {
+    return this.map.get(key)?.get(subKey);
+  }
+
+  has(key, subKey) {
+    const subMap = this.map.get(key);
+    return subMap && subMap.has(subKey);
+  }
+
+  delete(key, subKey) {
+    const subMap = this.map.get(key);
+    if (subMap) subMap.delete(subKey);
+  }
+
+  clear() {
+    for (const subMap of this.map.values())
+      subMap.clear();
+    this.map.clear();
+  }
+
+  *keys() {
+    for (const [key, subMap] of this.map)
+    for (const subKey of subMap.keys())
+      yield [key, subKey];
+  }
+
+  *values() {
+    for (const subMap of this.map.values())
+    for (const value of subMap.values())
+      yield value;
+  }
+
+  *entries() {
+    for (const [key, subMap] of this.map)
+    for (const [subKey, value] of subMap)
+      yield [key, subKey, value];
+  }
+
+  [Symbol.iterator]() {
+    return this.entries();
+  };
+}

mylogger.js

@@ -4,6 +4,7 @@ const ZongJi = require('./zongji');
 const mysql = require('mysql2/promise');
 const {loadConfig} = require('./lib/util');
 const ModelLoader = require('./lib/model-loader');
+const MultiMap = require('./lib/multi-map');
 
 module.exports = class MyLogger {
   constructor() {
@@ -18,12 +19,10 @@ module.exports = class MyLogger {
 
   async start() {
     const conf = this.conf = loadConfig(__dirname, 'config');
-
-    const {logMap, schemaMap} = this.modelLoader.init(conf);
-    Object.assign(this, {logMap, schemaMap});
+    Object.assign(this, this.modelLoader.init(conf));
 
     const includeSchema = {};
-    for (const [schemaName, tableMap] of this.schemaMap)
+    for (const [schemaName, tableMap] of this.schemaMap.map)
       includeSchema[schemaName] = Array.from(tableMap.keys());
   
     this.zongjiOpts = {
@@ -64,7 +63,7 @@ module.exports = class MyLogger {
 
     for (const logInfo of this.logMap.values()) {
       const table = logInfo.table;
-      const sqlTable = `${db.escapeId(table.schema)}.${db.escapeId(table.name)}`
+      const sqlTable = `${db.escapeId(table.schema)}.${db.escapeId(table.name)}`;
       logInfo.addStmt = await db.prepare(
         `INSERT INTO ${sqlTable}
           SET originFk = ?,
@@ -95,7 +94,10 @@ module.exports = class MyLogger {
       );
     }
 
-    await this.modelLoader.loadSchema(this.schemaMap, db);
+    await this.modelLoader.loadSchema(db,
+      this.schemaMap,
+      this.showTables
+    );
 
     // Zongji
 
@@ -306,8 +308,7 @@ module.exports = class MyLogger {
   onRowEvent(evt, eventName) {
     const table = evt.tableMap[evt.tableId];
     const tableName = table.tableName;
-    const tableInfo = this.schemaMap
-      .get(table.parentSchema)?.get(tableName);
+    const tableInfo = this.schemaMap.get(table.parentSchema, tableName);
     if (!tableInfo) return;
 
     const action = actions[eventName];
@@ -412,17 +413,20 @@ module.exports = class MyLogger {
           const ops = [];
           let txStarted;
           try {
+            for (let i = 0; i < conf.maxBulkLog && queue.length; i++)
+              ops.push(queue.shift());
+
+            await this.getShowValues(ops);
+
             await db.query('START TRANSACTION');
             txStarted = true;
   
-            for (let i = 0; i < conf.maxBulkLog && queue.length; i++) {
-              op = queue.shift();
-              ops.push(op);
+            for (op of ops)
               await this.applyOp(op);
-            }
 
             this.debug('Queue', `applied: ${ops.length}, remaining: ${queue.length}`);
-            await this.savePosition(op.binlogName, op.evt.nextPosition)
+            await this.savePosition(op.binlogName, op.evt.nextPosition);
+
             await db.query('COMMIT');
           } catch(err) {
             queue.unshift(...ops);
@@ -449,6 +453,72 @@ module.exports = class MyLogger {
     }
   }
 
+  async getShowValues(ops) {
+    const {db, showTables} = this;
+    const showValues = new MultiMap();
+
+    // Fetch relations id
+
+    for (const op of ops) {
+      if (op.hasShowValues) continue;
+      const {relations} = op.tableInfo;
+      for (const change of op.changes) {
+        if (op.action == 'update') {
+          getRelationsId(relations, change.newI);
+          getRelationsId(relations, change.oldI);
+        } else
+          getRelationsId(relations, change.instance);
+      }
+    }
+
+    function getRelationsId(relations, row) {
+      for (const col in row) {
+        const relation = relations.get(col);
+        if (!relation) continue;
+        const {schema, table} = relation;
+        let ids = showValues.get(schema, table);
+        if (!ids) showValues.set(schema, table, ids = new Map());
+        if (!ids.has(row[col])) ids.set(row[col], null);
+      }
+    }
+
+    // Fetch show values
+
+    for (const [schema, table, ids] of showValues) {
+      const tableInfo = showTables.get(schema, table);
+      const [res] = await db.query(
+        tableInfo.selectStmt,
+        [Array.from(ids.keys())]
+      );
+      for (const row of res)
+        ids.set(row.id, row.val);
+    }
+
+    // Fill rows with show values
+
+    for (const op of ops) {
+      const {relations} = op.tableInfo;
+      for (const change of op.changes) {
+        if (op.action == 'update') {
+          setShowValues(relations, change.newI);
+          setShowValues(relations, change.oldI);
+        } else
+          setShowValues(relations, change.instance);
+      }
+      op.hasShowValues = true;
+    }
+
+    function setShowValues(relations, row) {
+      for (const col in row) {
+        const relation = relations.get(col);
+        if (!relation) continue;
+        const {schema, table} = relation;
+        const showValue = showValues.get(schema, table).get(row[col]);
+        if (showValue) row[`${col}$`] = showValue;
+      }
+    }
+  }
+
   async applyOp(op) {
     const {conf} = this;
     const {
@@ -461,7 +531,7 @@ module.exports = class MyLogger {
     const logInfo = tableInfo.log;
     const isDelete = action == 'delete';
     const isUpdate = action == 'update';
-    const isMain = tableInfo.isMain;
+    const isSecondary = !tableInfo.isMain;
     const relation = tableInfo.relation;
 
     for (const change of changes) {
@@ -484,12 +554,12 @@ module.exports = class MyLogger {
       const created = new Date(evt.timestamp);
       const modelName = tableInfo.modelName;
       const modelId = row[tableInfo.idName];
-      const modelValue = tableInfo.showField && !isMain
+      const modelValue = tableInfo.showField && isSecondary
         ? row[tableInfo.showField] || null
         : null;
       const oldInstance = oldI ? JSON.stringify(oldI) : null;
-      const originFk = !isMain ? row[relation] : modelId;
-      const originChanged = isUpdate && !isMain
+      const originFk = isSecondary ? row[relation] : modelId;
+      const originChanged = isUpdate && isSecondary
         && newI[relation] !== undefined;
 
       let deleteRow;

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "mylogger",
-  "version": "0.1.21",
+  "version": "0.1.22",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "mylogger",
-      "version": "0.1.21",
+      "version": "0.1.22",
       "license": "GPL-3.0",
       "dependencies": {
         "colors": "^1.4.0",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "mylogger",
-  "version": "0.1.21",
+  "version": "0.1.22",
   "author": "Verdnatura Levante SL",
   "description": "MySQL and MariaDB logger using binary log",
   "license": "GPL-3.0",