Support UUID x Text comparison

[MattDelac]

Describe the bug
Matt Delac 6:38 PM
Hi
@Erez Shinan, follow up of my PR.
It sounds like we also need to support UUID x Text?

TypeError: Incompatible types for column 'owner_id': String_UUID() <-> Text()

On my use case, it happens between MySQL UUID <> Redshift Text.
I am not sure where UUID fits the class Dialect object though

Describe the environment

Describe which OS you're using, which sqeleton version, and any other information that might be relevant to this bug.

[timothyhinh79]

Any update on this issue? I'm seeing this happen on Postgres where the column is being converted to "String_UUID" in one table, but is classified as "Text" in the other table due to detection of non-UUID values.

[dlawin]

Any update on this issue? I'm seeing this happen on Postgres where the column is being converted to "String_UUID" in one table, but is classified as "Text" in the other table due to detection of non-UUID values.

Hi @timothyhinh79, we're not currently working on this but may in the future, in the meantime we'd happily accept a community contribution

[ramdharam]

@dlawin - I had this exact issue when comparing snowflake and postgres and I was able to come up with a simple approach that will overcome this issue in my fork. May be that can help. I will push it out and hopefully it can overcome your issue too.

[github-actions]

This issue has been marked as stale because it has been open for 60 days with no activity. If you would like the issue to remain open, please comment on the issue and it will be added to the triage queue. Otherwise, it will be closed in 7 days.

[github-actions]

Although we are closing this issue as stale, it's not gone forever. Issues can be reopened if there is renewed community interest. Just add a comment and it will be reopened for triage.

[MattDelac]

@dlawin - I had this exact issue when comparing snowflake and postgres and I was able to come up with a simple approach that will overcome this issue in my fork. May be that can help. I will push it out and hopefully it can overcome your issue too.

Hey did you get that working?