[KYUUBI #7109] Ignore the ? in backticks

[ruanwenjun]

Why are the changes needed?

We will split the sql by ? when we use KyuubiPreparedStatement. But there exist corner case when ? exist in backticks.
For example, below sql contains ?, but we shouldn't split it by ?.

SELECT `(ds|hr)?+.+` FROM sales

More details can find at https://hive.apache.org/docs/latest/languagemanual-select_27362043/#regex-column-specification

Hive upstream fix - HIVE-29060

How was this patch tested?

UT.

Was this patch authored or co-authored using generative AI tooling?

NO.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (e98ad7b) to head (7140980).

Files with missing lines	Patch %	Lines
...c/main/java/org/apache/kyuubi/jdbc/hive/Utils.java	0.00%	4 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff           @@
##           master   #7125   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         700     700           
  Lines       43374   43377    +3     
  Branches     5873    5875    +2     
======================================
- Misses      43374   43377    +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[pan3793]

Kyuubi JDBC Driver is derived from Hive JDBC Driver, was this fixed in Hive project?

[ruanwenjun]

Kyuubi JDBC Driver is derived from Hive JDBC Driver, was this fixed in Hive project?

Hive doesn't have a fix for this. 😢

[pan3793]

What about spark-sql? The ported JDBC driver code does not have sufficient test coverage, we should be careful when touching this code

[ruanwenjun]

What about spark-sql? The ported JDBC driver code does not have sufficient test coverage, we should be careful when touching this code

spark-sql supports this if we set spark.sql.parser.quotedRegexColumnNames=true. I agree with you — it's dangerous and might introduce bugs.

Actually, my initial idea was to get it merged into the community first, and then cherry-pick it into our version. Going through the community review process can help reduce the likelihood of bugs.

[pan3793]

let me take a look at how spark-sql processes this.

[ruanwenjun]

I create jira to Hive: https://issues.apache.org/jira/browse/HIVE-29060?page=com.atlassian.jira.plugin.system.issuetabpanels%3Aall-tabpanel

Seems they begin to deal with this.

[ruanwenjun]

Hive has fixed this yesterday apache/hive@9a68ebc @pan3793

[pan3793]

Thanks, merged to master/1.10