Remove use of multipleOf in dqx-yaml

[gummiorri]

Summary by cubic

Removed handling of multipleOf in the dqxyaml translator so DQX no longer emits SQL modulo checks for this constraint. Aligns with Linear #98 and simplifies schema-to-check translation.

• Refactors
  • Dropped multipleOf SQL check in collectChecks and fixed lint.
  • Removed TestTranslate_MultipleOf.

^{Written for commit e1f4849. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 2 files

Architecture diagram

sequenceDiagram
    participant Client as Application/CLI
    participant Translator as dqxyaml.Translator
    participant Collector as collectChecks()
    participant Engine as DQX Engine
    participant DB as Data Warehouse

    Note over Client,DB: Schema to Data Quality Check Flow

    Client->>Translator: translateSchema(jsonSchema)
    
    loop For each Property in Schema
        Translator->>Collector: collectChecks(colName, constraints)
        
        opt Constraint: Required
            Collector->>Collector: Append "is_not_null"
        end
        
        opt Constraint: Array (MinItems/MaxItems)
            Collector->>Collector: Append "sql_expression" (size check)
        end

        Note over Collector: CHANGED: MultipleOf constraint is no<br/>longer processed into SQL modulo checks.

        Collector-->>Translator: return []dqxCheck
    end

    Translator-->>Client: DQX Configuration (YAML)

    Client->>Engine: Run Quality Checks
    
    loop For each Check
        Engine->>DB: Execute SQL (is_not_null, size(), etc.)
        DB-->>Engine: Result
    end
    
    Engine-->>Client: Validation Report

Loading

[giuseppegrieco]

LGTM