doris icon indicating copy to clipboard operation
doris copied to clipboard

[improvement](jdbc catalog) remove useless jdbc catalog code

Open zy-kkk opened this issue 9 months ago • 10 comments

Proposed changes

Issue Number: close #xxx

Since JdbcExecutor splitting for different data sources has been completed, I deleted the old JdbcExecutor code in this PR, and since the current reading of different data types is processed by each data source, we do not need to continue in BE Internal inspection is performed, so this part of the code is also deleted.In addition, since support for the nebula database Jdbc appearance is no longer needed, this part of the code has also been removed.

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

zy-kkk avatar May 16 '24 16:05 zy-kkk

Thank you for your contribution to Apache Doris. Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website. See Doris Document.

doris-robot avatar May 16 '24 16:05 doris-robot

run buildall

zy-kkk avatar May 16 '24 16:05 zy-kkk

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 16 '24 17:05 github-actions[bot]

TeamCity be ut coverage result: Function Coverage: 35.71% (9004/25216) Line Coverage: 27.39% (74495/271961) Region Coverage: 26.63% (38501/144557) Branch Coverage: 23.47% (19643/83700) Coverage Report: http://coverage.selectdb-in.cc/coverage/62ec37e8f71acfbb3b30f4e2ba2a0a771fa71c37_62ec37e8f71acfbb3b30f4e2ba2a0a771fa71c37/report/index.html

doris-robot avatar May 16 '24 18:05 doris-robot

run buildall

zy-kkk avatar May 17 '24 03:05 zy-kkk

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 17 '24 03:05 github-actions[bot]

TeamCity be ut coverage result: Function Coverage: 35.70% (9005/25224) Line Coverage: 27.39% (74516/272070) Region Coverage: 26.63% (38519/144618) Branch Coverage: 23.48% (19662/83732) Coverage Report: http://coverage.selectdb-in.cc/coverage/2e6217d66557bedd26eb0d68bef2b1169760ae20_2e6217d66557bedd26eb0d68bef2b1169760ae20/report/index.html

doris-robot avatar May 17 '24 05:05 doris-robot

run buildall

zy-kkk avatar May 17 '24 15:05 zy-kkk

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 17 '24 15:05 github-actions[bot]

TeamCity be ut coverage result: Function Coverage: 35.70% (9006/25225) Line Coverage: 27.37% (74474/272077) Region Coverage: 26.62% (38512/144652) Branch Coverage: 23.47% (19654/83754) Coverage Report: http://coverage.selectdb-in.cc/coverage/b715f02c8a2c06c0dd41ad63cfc03c0ebbc8d4a1_b715f02c8a2c06c0dd41ad63cfc03c0ebbc8d4a1/report/index.html

doris-robot avatar May 17 '24 17:05 doris-robot