DataJunction / dj

A metrics platform.
http://datajunction.io
MIT License
34 stars 15 forks source link

Attempt to fix the comparison for struct types. #1009

Closed agorajek closed 4 months ago

agorajek commented 4 months ago

Summary

This fixes comparison for struct types that we ingest from our repo. Another way to fix it would be to try to match the other side but I'd like to move on for now since things seems to be working.

Test Plan

Unit tests plus internal test on a mock source node:

create table olek.test_1 (a string, b int, c double, d map<string, string>, e array<int>, s struct<a int, b string>);

Deployment Plan

n/a

netlify[bot] commented 4 months ago

Deploy Preview for thriving-cassata-78ae72 canceled.

Name Link
Latest commit 3a5149fb9cb885b01939046847c8ab6a0164a3fd
Latest deploy log https://app.netlify.com/sites/thriving-cassata-78ae72/deploys/664d893ead26410008abb295