【Bug】 OracleSouce edited again, ConnectionType is null

[AliceXiaoLu]

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

[AliceXiaoLu]

@caishunfeng please review

[codecov-commenter]

Codecov Report

Merging #13881 (49eda1a) into dev (7ee66f2) will increase coverage by 0.00%.
The diff coverage is 0.00%.

❗ Current head 49eda1a differs from pull request most recent head 0404e0f. Consider uploading reports for the commit 0404e0f to get more accurate results

@@            Coverage Diff            @@
##                dev   #13881   +/-   ##
=========================================
  Coverage     38.86%   38.86%           
+ Complexity     4457     4454    -3     
=========================================
  Files          1158     1158           
  Lines         42424    42425    +1     
  Branches       4772     4776    +4     
=========================================
+ Hits          16487    16488    +1     
+ Misses        24123    24122    -1     
- Partials       1814     1815    +1     

Impacted Files	Coverage Δ
...source/oracle/param/OracleDataSourceProcessor.java	48.14% <0.00%> (-0.91%)	⬇️

... and 4 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[rickchengx]

Hi, @AliceXiaoLu , thanks for your work. Could you please update the related UT?

[AliceXiaoLu]

Hi, @AliceXiaoLu , thanks for your work. Could you please update the related UT?

What exactly is UT? For example???

[rickchengx]

Hi, @AliceXiaoLu , thanks for your work. Could you please update the related UT?

What exactly is UT? For example???

Maybe you could check and update the OracleDataSourceProcessorTest to test the case of this PR :)

[AliceXiaoLu]

Hi, @AliceXiaoLu , thanks for your work. Could you please update the related UT?

What exactly is UT? For example???

Maybe you could check and update the OracleDataSourceProcessorTest to test the case of this PR :)

Yes， OracleDataSourceProcessorTest has this case. and is right。

[AliceXiaoLu]

@caishunfeng Please help me to add Label,Thank you. Lable must include one of the following: feature, bug, improvement, document, chore AND Label must include one or more version numbers.

[AliceXiaoLu]

@caishunfeng +1

[AliceXiaoLu]

@rickchengx Please help to review

[rickchengx]

@rickchengx Please help to review

• LGTM, please check the failed CI
• And could you please add the related issue link in the PR, E.g., close: #issue

cc @caishunfeng

[SbloodyS]

+1