Skip to content
  • sundar-mudupalli-work's avatar
    506d8db5
    fix: Only strip primary keys when they are fixed chars, not for varchars (#1472) · 506d8db5
    sundar-mudupalli-work authored
    
    
    * Updated to support Teradata, Oracle and others
    
    * How to handle strings and encodings
    
    * Updated String Processing
    
    * Fixed existing tests for Postgres, Oracle
    
    * Fixed teradata validation issue
    
    * Added Fixed Char and Variable Char tables
    
    * Updated to mostly remove trim_string_pks - test_config_manager failing
    
    * Updated cli_tools.py
    
    * Updated to use Neil's suggestion regarding storing raw data types in config manager
    
    * Created fixed_char and varchar tests, fixed unit tests
    
    * Updated documentation to remove tsp
    
    * Updated different clients add is_char_type_padded function
    
    * Updated line count on yaml files; The property trim_string_pks was removed, resulting in one fewer line output; No harm in reading that property from old yaml files, it is no longer used
    
    * updated to migrate backend specific stuff to backends
    
    * Updated to fix unit tests
    
    * Updated Internal Design doc
    
    * Removed table definitions in sql files
    
    * Added warning if yaml files still have the trim string primary keys flag turned on
    
    * Made changes for Random Row and Partition Builder
    
    * Renamed function to is_padded_char
    
    * Updated to use is_padded_char rather than _is_padded_char
    
    * feat: Add PaddedCharLength rule for backends with is_char_type_padded method
    
    * Linted
    
    * Linted
    
    * Linted
    
    * Updated documentation
    
    * docs: Update product names in strings.md
    
    * Fixed tests on mysql, postgres, teradata
    
    * Fixed Random rows Oracle, queries and tables will have padded primary keys stripped
    
    * Updating Typing to fix errors
    
    * Linted
    
    * Updated tests for clarity
    
    ---------
    
    Co-authored-by: default avatarHelen Cristina <helensilva14@gmail.com>
    Co-authored-by: default avatarNeil Johnson <neiljohnson@google.com>
    506d8db5
    fix: Only strip primary keys when they are fixed chars, not for varchars (#1472)
    sundar-mudupalli-work authored
    
    
    * Updated to support Teradata, Oracle and others
    
    * How to handle strings and encodings
    
    * Updated String Processing
    
    * Fixed existing tests for Postgres, Oracle
    
    * Fixed teradata validation issue
    
    * Added Fixed Char and Variable Char tables
    
    * Updated to mostly remove trim_string_pks - test_config_manager failing
    
    * Updated cli_tools.py
    
    * Updated to use Neil's suggestion regarding storing raw data types in config manager
    
    * Created fixed_char and varchar tests, fixed unit tests
    
    * Updated documentation to remove tsp
    
    * Updated different clients add is_char_type_padded function
    
    * Updated line count on yaml files; The property trim_string_pks was removed, resulting in one fewer line output; No harm in reading that property from old yaml files, it is no longer used
    
    * updated to migrate backend specific stuff to backends
    
    * Updated to fix unit tests
    
    * Updated Internal Design doc
    
    * Removed table definitions in sql files
    
    * Added warning if yaml files still have the trim string primary keys flag turned on
    
    * Made changes for Random Row and Partition Builder
    
    * Renamed function to is_padded_char
    
    * Updated to use is_padded_char rather than _is_padded_char
    
    * feat: Add PaddedCharLength rule for backends with is_char_type_padded method
    
    * Linted
    
    * Linted
    
    * Linted
    
    * Updated documentation
    
    * docs: Update product names in strings.md
    
    * Fixed tests on mysql, postgres, teradata
    
    * Fixed Random rows Oracle, queries and tables will have padded primary keys stripped
    
    * Updating Typing to fix errors
    
    * Linted
    
    * Updated tests for clarity
    
    ---------
    
    Co-authored-by: default avatarHelen Cristina <helensilva14@gmail.com>
    Co-authored-by: default avatarNeil Johnson <neiljohnson@google.com>
Loading