• James Bartlett's avatar
    Use ST_NONE instead of ST_UNSPECIFIED as default. · 82c5d325
    James Bartlett authored
    Summary: @philkuz pointed out that the intention of ST_UNSPECIFIED was to catch bugs where the semantic type wasn't set properly. This diff changes the system to treat ST_NONE as the default semantic type and catch all for inference rules. This surfaced a bug where the Metadata service wasn't correctly propagating Semantic type information from stirling (even though this information doesn't exist in stirling yet, it wasn't correctly propagating the ST_NONEs).
    
    Test Plan: Checked that all output columns in the UI now have sem type ST_NONE and not ST_UNSPECIFIED.
    
    Reviewers: philkuz, nserrino, #engineering
    
    Reviewed By: nserrino, #engineering
    
    Subscribers: philkuz
    
    Differential Revision: https://phab.corp.pixielabs.ai/D5308
    
    GitOrigin-RevId: 691b725a74819c1b9a7097bae208ba19fd12116a
    82c5d325