[AMORO-3989] Amoro's support for the domestic database Damang

[linjianchang]

[AMORO-3989] Amoro's support for the domestic database Damang

[xxubai]

Can you fix the unit tests

[codecov-commenter]

Codecov Report

❌ Patch coverage is 36.36364% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 7.80%. Comparing base (cbdc517) to head (6b2aa2d).
⚠️ Report is 15 commits behind head on master.

Files with missing lines	Patch %	Lines
...he/amoro/server/persistence/DataSourceFactory.java	0.00%	5 Missing and 2 partials ⚠️

❗ There is a different number of reports uploaded between BASE (cbdc517) and HEAD (6b2aa2d). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (cbdc517)	HEAD (6b2aa2d)
trino	1	0

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #3992       +/-   ##
============================================
- Coverage     22.12%   7.80%   -14.33%     
+ Complexity     2461     895     -1566     
============================================
  Files           445     639      +194     
  Lines         40897   51508    +10611     
  Branches       5767    6545      +778     
============================================
- Hits           9050    4019     -5031     
- Misses        31089   47233    +16144     
+ Partials        758     256      -502     

Flag	Coverage Δ
core	7.80% <36.36%> (?)
trino	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

[xxubai]

It should be provided

[linjianchang]

It should be provided

I see that the JDBC driver packages for PostgreSQL and Derby are not provided

[xxubai]

I don’t think Dameng needs to be included in the release. Similar to MySQL, users who need it can add the JDBC dependency manually, rather than having it as a standard component.

[linjianchang]

I don’t think Dameng needs to be included in the release. Similar to MySQL, users who need it can add the JDBC dependency manually, rather than having it as a standard component.

Aleady modified!

[linjianchang]

@xxubai Thanks for your review. Do you have any other suggestions

[xxubai]

Adding support for other database connectors is technically straightforward. However, the primary concern is long-term maintenance. Is there sufficient community interest and commitment to help maintain this connector over time? I would like to invite more community members to participate in the discussion. @zhoujinsong @czy006 @Aireed

[github-actions]

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions.