fix: handle INTEGER/BOOLEAN/TIMESTAMP type names in DML — INFORMATION_SCHEMA returns different names than table resource API

Author: datlechin

Plugins/BigQueryDriverPlugin/BigQueryPluginDriver.swift

@@ -694,6 +694,7 @@ internal final class BigQueryPluginDriver: PluginDatabaseDriver, @unchecked Send
         guard let conn = connection else { return nil }
 
         let dataset = lock.withLock { _currentDataset } ?? ""
+        Self.logger.info("generateStatements: dataset='\(dataset, privacy: .public)' table='\(table, privacy: .public)' self=\(ObjectIdentifier(self).debugDescription, privacy: .public)")
 
         // Block DML on external tables
         let tableType: String? = lock.withLock {
@@ -717,6 +718,8 @@ internal final class BigQueryPluginDriver: PluginDatabaseDriver, @unchecked Send
             return columns.map { _ in "STRING" }
         }
 
+        Self.logger.info("generateStatements: typeNames=\(typeNames, privacy: .public)")
+
         let generator = BigQueryStatementGenerator(
             projectId: conn.projectId,
             dataset: dataset,

Plugins/BigQueryDriverPlugin/BigQueryStatementGenerator.swift

@@ -162,7 +162,8 @@ internal struct BigQueryStatementGenerator {
 
         let upperType = typeName.uppercased()
 
-        if upperType == "INT64" || upperType == "FLOAT64" ||
+        if upperType == "INT64" || upperType == "INTEGER" ||
+            upperType == "FLOAT64" || upperType == "FLOAT" ||
             upperType == "NUMERIC" || upperType == "BIGNUMERIC"
         {
             let isNumeric = value.range(
@@ -173,7 +174,7 @@ internal struct BigQueryStatementGenerator {
             return "'\(escapeString(value))'"
         }
 
-        if upperType == "BOOL" {
+        if upperType == "BOOL" || upperType == "BOOLEAN" {
             let lower = value.lowercased()
             if lower == "true" || lower == "1" { return "TRUE" }
             if lower == "false" || lower == "0" { return "FALSE" }
@@ -204,6 +205,20 @@ internal struct BigQueryStatementGenerator {
             return "FROM_BASE64('\(escapeString(value))')"
         }
 
+        // Temporal types need explicit casting for WHERE clause comparisons
+        if upperType == "TIMESTAMP" {
+            return "TIMESTAMP '\(escapeString(value))'"
+        }
+        if upperType == "DATE" {
+            return "DATE '\(escapeString(value))'"
+        }
+        if upperType == "DATETIME" {
+            return "DATETIME '\(escapeString(value))'"
+        }
+        if upperType == "TIME" {
+            return "TIME '\(escapeString(value))'"
+        }
+
         return "'\(escapeString(value))'"
     }