JDBC Catalog:Change SQL like escape character from '\\' to '!'

[xuchuanqiu]

JDBC Catalog:Change the escape character "" to "!", to solve the problem that in MySQL the escaped '' must be written as '\';

[nastra]

fixes #10056

@xuchuanqiu can you rebase this one please to fix the merge conflicts?

[jbonofre]

Does it work with any backend like PostgreSQL ?

[jbonofre]

Tested without problem with PostgreSQL

[jbonofre]

I didn't know this ! notation. I will test on PostgreSQL and MySQL to be sure it works with these vendors.

[jbonofre]

If @xuchuanqiu doesn't have the bandwidth, I can help on this one. If it works with MySQL, SQLlite and PostgreSQL, it's a nice fix to include in Iceberg 1.5.1.

[amogh-jahagirdar]

I've added this to the 1.5.1 milestone. If we validate that it works on all the backends as expected, then I think we can include in the patch release (we'll also discuss this in the community sync today).

[jbonofre]

I tested with MySQL 8.3.0 (both server and JDBC driver) and I confirm the ! notation works.
I used MySQL 8.3.0 docker image, create the schema and updated the GET_NAMESPACE_SQL statement.
No SQL statement error, and the namespace is retrieved without problem.

I'm doing the same test with PostgreSQL now, to be sure the same statement works fine with PostgreSQL.

[jbonofre]

I also tested with PostgreSQL 42.7.3 and it works fine.

In order to fix the conflict, I propose to create a clean PR.

[jbonofre]

Replaced by #10167 to merge the conflict.

[amogh-jahagirdar]

Thanks @jbonofre ! Discussed offline, we'll ensure that @xuchuanqiu gets co-authorship/attribution. Thanks @xuchuanqiu

[amogh-jahagirdar]

I'll close this since #9407 has been merged, thanks @xuchuanqiu @jbonofre !