[KYUUBI #310] GetColumns supports DSv2 and keeps its backward compatibility

[yaooqinn]

❨?❩

close #310

Why are the changes needed?

How was this patch tested?

• Add some test cases that check the changes thoroughly including negative and positive cases if possible

• Add screenshots for manual tests if appropriate

• Run test locally before make a pull request

[codecov]

Codecov Report

Merging #358 (d332be5) into master (54c306b) will increase coverage by 0.65%.
The diff coverage is 88.80%.

@@            Coverage Diff             @@
##           master     #358      +/-   ##
==========================================
+ Coverage   80.14%   80.79%   +0.65%     
==========================================
  Files          98       98              
  Lines        3535     3557      +22     
  Branches      412      412              
==========================================
+ Hits         2833     2874      +41     
+ Misses        490      469      -21     
- Partials      212      214       +2     

Impacted Files	Coverage Δ
...he/kyuubi/engine/spark/shim/CatalogShim_v3_0.scala	85.13% <75.00%> (ø)
...he/kyuubi/engine/spark/shim/CatalogShim_v2_4.scala	76.78% <75.75%> (ø)
...he/kyuubi/engine/spark/shim/SparkCatalogShim.scala	79.16% <93.10%> (ø)
...he/kyuubi/engine/spark/operation/GetCatalogs.scala	83.33% <100.00%> (ø)
...che/kyuubi/engine/spark/operation/GetColumns.scala	97.95% <100.00%> (+3.97%)	⬆️
...che/kyuubi/engine/spark/operation/GetSchemas.scala	90.00% <100.00%> (ø)
.../kyuubi/engine/spark/operation/GetTableTypes.scala	100.00% <100.00%> (ø)
...ache/kyuubi/engine/spark/operation/GetTables.scala	97.14% <100.00%> (ø)
...ine/spark/operation/SparkSQLOperationManager.scala	91.89% <100.00%> (ø)
.../scala/org/apache/kyuubi/schema/SchemaHelper.scala	98.78% <100.00%> (+1.05%)	⬆️
... and 2 more

[github-actions]

Scan Summary

Tool	Critical	High	Medium	Low	Status
Security Audit for Infrastructure	0	0	0	0	✅
Java Source Analyzer	0	0	0	1	✅
Class File Analyzer	0	6	0	0	❌
Python Security Analysis	3	0	0	0	❌
Python Source Analyzer	0	0	0	0	✅
Scala Security Audit	0	6	0	0	❌
Shell Script Analysis	0	0	0	2	✅

Recommendation

Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍

[github-actions]

Scan Summary

Tool	Critical	High	Medium	Low	Status
Dependency Scan (java)	0	2	5	0	✅
Security Audit for Infrastructure	0	0	0	0	✅
Java Source Analyzer	0	0	0	1	✅
Class File Analyzer	0	6	0	0	❌
Python Security Analysis	3	0	0	0	❌
Python Source Analyzer	0	0	0	0	✅
Scala Security Audit	0	6	0	0	❌
Shell Script Analysis	0	0	0	0	✅

Recommendation

Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍