Redshift support for multibyte UTF-8 identifiers

Author: yoavcloud

src/dialect/redshift.rs

@@ -82,13 +82,13 @@ impl Dialect for RedshiftSqlDialect {
     fn is_identifier_start(&self, ch: char) -> bool {
         // Extends Postgres dialect with sharp and UTF-8 multibyte chars
         // https://docs.aws.amazon.com/redshift/latest/dg/r_names.html
-        PostgreSqlDialect {}.is_identifier_start(ch) || ch == '#' || ch as u32 > 0x7F
+        PostgreSqlDialect {}.is_identifier_start(ch) || ch == '#' || !ch.is_ascii()
     }
 
     fn is_identifier_part(&self, ch: char) -> bool {
         // Extends Postgres dialect with sharp and UTF-8 multibyte chars
         // https://docs.aws.amazon.com/redshift/latest/dg/r_names.html
-        PostgreSqlDialect {}.is_identifier_part(ch) || ch == '#' || ch as u32 > 0x7F
+        PostgreSqlDialect {}.is_identifier_part(ch) || ch == '#' || !ch.is_ascii()
     }
 
     /// redshift has `CONVERT(type, value)` instead of `CONVERT(value, type)`