Invalid tmp table name when using: unique_tmp_table_suffix

[atommych]

Description

When using unique_tmp_table_suffix=True, the UUID added generated contain dashes which causes the following error:
mismatched input '-'. Expecting: '(', '.', 'AS', 'COMMENT', 'WITH'

Checklist

• You followed contributing section
• You kept your Pull Request small and focused on a single feature or bug fix.
• You added unit testing when necessary
• You added functional testing when necessary

[nicor88]

@atommych could you have a look at the interagration tests? They might need to be readapted

[nicor88]

@atommych any plan to fix the integration tests? I believe that this regex must be re-adapted to omit the -, also if it's easier, we could consider to use replace('-','_') then in the regex for the integration tests we can simply replace the - with _. Using _ in the table name is still valid.

[atommych]

Hi @nicor88 , I will work on it till Friday! Thanks for the tips

[atommych]

Hi @nicor88, the integration test are good now!

[nicor88]

@colin-rogers-dbt and @mikealfare could you please have a look? this looks good to be merged.

@atommych update your branch with main then we are ready to go.