Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
AshinGau merged PR #26317: URL: https://github.com/apache/doris/pull/26317 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1807399451 TeamCity be ut coverage result: Function Coverage: 36.77% (8404/22853) Line Coverage: 29.34% (68205/232474) Region Coverage: 27.92% (35221/126143) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1807383150 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 44.71 seconds stream load tsv: 555 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1807376314 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 25c49242d9cab892615269e9bf26c7dbb3684cfa, data reload: false

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1807321635 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1807154493 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 44.48 seconds stream load tsv: 551 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1807151907 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 52a9f0c83ba59e09d724c2cb01afdc04f8cce7dc, data reload: false

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-12 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1807101332 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805484498 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 44.93 seconds stream load tsv: 560 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805480614 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 52a9f0c83ba59e09d724c2cb01afdc04f8cce7dc, data reload: false

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805481712 TeamCity be ut coverage result: Function Coverage: 36.76% (8403/22856) Line Coverage: 29.32% (68130/232358) Region Coverage: 27.93% (35223/126121) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-10 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805437446 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-10 Thread via GitHub
github-actions[bot] commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805324335 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805273281 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 52a9f0c83ba59e09d724c2cb01afdc04f8cce7dc, data reload: false

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-10 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805272536 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 44.6 seconds stream load tsv: 556 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-09 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805254399 TeamCity be ut coverage result: Function Coverage: 36.77% (8404/22856) Line Coverage: 29.33% (68149/232358) Region Coverage: 27.93% (35231/126121) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-09 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1805227938 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1803258229 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.06 seconds stream load tsv: 553 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1803241335 TeamCity be ut coverage result: Function Coverage: 36.75% (8401/22857) Line Coverage: 29.29% (68122/232585) Region Coverage: 27.91% (35222/126194) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
BePPPower commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1803218693 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1801505384 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.68 seconds stream load tsv: 554 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1801416500 TeamCity be ut coverage result: Function Coverage: 36.76% (8401/22856) Line Coverage: 29.29% (68113/232529) Region Coverage: 27.94% (35255/126162) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1801413754 TeamCity be ut coverage result: Function Coverage: 36.70% (8388/22856) Line Coverage: 29.26% (68027/232529) Region Coverage: 27.89% (35185/126162) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1801371602 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
zy-kkk commented on code in PR #26317: URL: https://github.com/apache/doris/pull/26317#discussion_r1386219050 ## fe/be-java-extensions/jdbc-scanner/src/main/java/org/apache/doris/jdbc/JdbcExecutor.java: ## @@ -483,1071 +362,402 @@ private void setValidationQuery(DruidDataSource

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1801347108 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-08 Thread via GitHub
zy-kkk commented on code in PR #26317: URL: https://github.com/apache/doris/pull/26317#discussion_r1386218804 ## fe/be-java-extensions/jdbc-scanner/src/main/java/org/apache/doris/jdbc/JdbcExecutor.java: ## @@ -483,1071 +362,402 @@ private void setValidationQuery(DruidDataSource

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-07 Thread via GitHub
morningman commented on code in PR #26317: URL: https://github.com/apache/doris/pull/26317#discussion_r1384767458 ## fe/be-java-extensions/jdbc-scanner/src/main/java/org/apache/doris/jdbc/JdbcExecutor.java: ## @@ -483,1071 +362,402 @@ private void

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-07 Thread via GitHub
github-actions[bot] commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1801192408 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-07 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1798177594 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 46.07 seconds stream load tsv: 557 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-07 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1798146967 TeamCity be ut coverage result: Function Coverage: 36.99% (8391/22684) Line Coverage: 29.45% (67923/230601) Region Coverage: 28.11% (35141/125025) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-07 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1798114135 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1797720652 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 46.05 seconds stream load tsv: 570 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1797696385 TeamCity be ut coverage result: Function Coverage: 36.99% (8390/22679) Line Coverage: 29.47% (67920/230467) Region Coverage: 28.12% (35140/124950) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1797626609 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794656582 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 46.56 seconds stream load tsv: 557 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
AshinGau commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794631862 Please finish the `Proposed changes` in detail. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
github-actions[bot] commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794630903 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
github-actions[bot] commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794630841 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
AshinGau commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794629750 LGTM -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794613158 TeamCity be ut coverage result: Function Coverage: 37.01% (8393/22677) Line Coverage: 29.48% (67927/230417) Region Coverage: 28.13% (35152/124944) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794564109 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794530063 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.49 seconds stream load tsv: 554 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794509298 TeamCity be ut coverage result: Function Coverage: 37.00% (8391/22677) Line Coverage: 29.47% (67902/230417) Region Coverage: 28.12% (35135/124944) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
github-actions[bot] commented on code in PR #26317: URL: https://github.com/apache/doris/pull/26317#discussion_r1383057871 ## be/src/vec/exec/vjdbc_connector.cpp: ## @@ -223,6 +198,158 @@ Status JdbcConnector::query() { return Status::OK(); } +Status

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-06 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1794467532 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-03 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1792178909 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.48 seconds stream load tsv: 574 seconds loaded 74807831229 Bytes,

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-03 Thread via GitHub
doris-robot commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1792161742 TeamCity be ut coverage result: Function Coverage: 37.29% (8451/22663) Line Coverage: 29.71% (68432/230304) Region Coverage: 28.35% (35411/124911) Branch Coverage:

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-03 Thread via GitHub
github-actions[bot] commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1792141171 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-03 Thread via GitHub
zy-kkk commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1792133118 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-02 Thread via GitHub
github-actions[bot] commented on PR #26317: URL: https://github.com/apache/doris/pull/26317#issuecomment-1790405179 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] [refactor](jni) unified jni framework for jdbc catalog [doris]

2023-11-02 Thread via GitHub
zy-kkk opened a new pull request, #26317: URL: https://github.com/apache/doris/pull/26317 ## Proposed changes Issue Number: close #xxx ## Further comments If this is a relatively large or complex change, kick off the discussion at