Re: [PR] [INLONG-9027][Manager] Fix SQL error in ClusterSetMapper.selectInlongStreamId [inlong]

2023-10-08 Thread via GitHub
dockerzhang merged PR #9028: URL: https://github.com/apache/inlong/pull/9028 -- 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: commits-unsubscr...@inlong.

[inlong] branch master updated: [INLONG-9027][Manager] Fix SQL error in ClusterSetMapper.selectInlongStreamId (#9028)

2023-10-08 Thread dockerzhang
This is an automated email from the ASF dual-hosted git repository. dockerzhang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/inlong.git The following commit(s) were added to refs/heads/master by this push: new 0db40a96ee [INLONG-9027][Manager] Fix SQL e

[inlong] branch master updated: [INLONG-9023][Manager] Fix the problem of failed to stop job task (#9024)

2023-10-08 Thread dockerzhang
This is an automated email from the ASF dual-hosted git repository. dockerzhang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/inlong.git The following commit(s) were added to refs/heads/master by this push: new 4802277803 [INLONG-9023][Manager] Fix the p

Re: [PR] [INLONG-9023][Manager] Fix the problem of failed to stop job task [inlong]

2023-10-08 Thread via GitHub
dockerzhang merged PR #9024: URL: https://github.com/apache/inlong/pull/9024 -- 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: commits-unsubscr...@inlong.

Re: [PR] [INLONG-8961][Manager] Server-side request forgery attack prevention in some classes [inlong]

2023-10-08 Thread via GitHub
fuweng11 commented on code in PR #9012: URL: https://github.com/apache/inlong/pull/9012#discussion_r1349656672 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/starrocks/StarRocksJdbcUtils.java: ## @@ -44,25 +45,50 @@ public class S

Re: [PR] [INLONG-8961][Manager] Server-side request forgery attack prevention in some classes [inlong]

2023-10-08 Thread via GitHub
fuweng11 commented on code in PR #9012: URL: https://github.com/apache/inlong/pull/9012#discussion_r1349656672 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/starrocks/StarRocksJdbcUtils.java: ## @@ -44,25 +45,50 @@ public class S

Re: [PR] [INLONG-8961][Manager] Server-side request forgery attack prevention in some classes [inlong]

2023-10-08 Thread via GitHub
fuweng11 commented on code in PR #9012: URL: https://github.com/apache/inlong/pull/9012#discussion_r1349657650 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/ck/ClickHouseJdbcUtils.java: ## @@ -39,37 +39,153 @@ public class Click

Re: [PR] [INLONG-8961][Manager] Server-side request forgery attack prevention in some classes [inlong]

2023-10-08 Thread via GitHub
fuweng11 commented on code in PR #9012: URL: https://github.com/apache/inlong/pull/9012#discussion_r1349657650 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/ck/ClickHouseJdbcUtils.java: ## @@ -39,37 +39,153 @@ public class Click

Re: [PR] [INLONG-9016][Dashboard] Change the wrapWithInlongMsg to wrapType in data stream [inlong]

2023-10-08 Thread via GitHub
healchow commented on code in PR #9019: URL: https://github.com/apache/inlong/pull/9019#discussion_r1349675687 ## inlong-dashboard/src/plugins/streams/common/StreamDefaultInfo.ts: ## @@ -262,24 +262,28 @@ export class StreamDefaultInfo implements DataWithBackend, RenderRow, Ren

[I] [Improve] update how-to-verify doc to use correct test command [inlong-website]

2023-10-08 Thread via GitHub
EMsnap opened a new issue, #873: URL: https://github.com/apache/inlong-website/issues/873 ### Description [Improve] update how-to-verify doc to use correct test command Mvn clean package would run test before install Sometimes we need to install first to run some tests

[PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin opened a new pull request, #9029: URL: https://github.com/apache/inlong/pull/9029 ### Prepare a Pull Request *(Change the title refer to the following example)* - Title Example: [INLONG-XYZ][Component] Title of the pull request *(The following *XYZ* should be replac

[PR] add hive connector for flink1.15 [inlong]

2023-10-08 Thread via GitHub
huasiy opened a new pull request, #9030: URL: https://github.com/apache/inlong/pull/9030 ### Prepare a Pull Request *(Change the title refer to the following example)* - Title Example: [INLONG-XYZ][Component] Title of the pull request *(The following *XYZ* should be replaced

[PR] [INLONG-870][Release] Update how-to-verify doc to use correct test command [inlong-website]

2023-10-08 Thread via GitHub
EMsnap opened a new pull request, #874: URL: https://github.com/apache/inlong-website/pull/874 ### Prepare a Pull Request - Fixes #873 ### Motivation Mvn clean package would run test before install Sometimes we need to install first to run some tests So install

[I] [Improve][Doc] Pick #869 and #863 to 1.9.0 [inlong-website]

2023-10-08 Thread via GitHub
dockerzhang opened a new issue, #875: URL: https://github.com/apache/inlong-website/issues/875 ### Description lack of #869 and #863 for 1.9.0 document branch ### Are you willing to submit PR? - [X] Yes, I am willing to submit a PR! ### Code of Conduct - [X]

[PR] [INLONG-875][Doc] Pick #869 and #863 to 1.9.0 [inlong-website]

2023-10-08 Thread via GitHub
dockerzhang opened a new pull request, #876: URL: https://github.com/apache/inlong-website/pull/876 ### Prepare a Pull Request *(Change the title refer to the following example)* - Title Example: [INLONG-XYZ][Component] Title of the pull request *(The following *XYZ* should b

Re: [PR] [INLONG-870][Sort] Update starrocks.md [inlong-website]

2023-10-08 Thread via GitHub
EMsnap merged PR #871: URL: https://github.com/apache/inlong-website/pull/871 -- 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: commits-unsubscr...@inlong

Re: [I] [Improve] Fix the document for the starrocks load node [inlong-website]

2023-10-08 Thread via GitHub
EMsnap closed issue #870: [Improve] Fix the document for the starrocks load node URL: https://github.com/apache/inlong-website/issues/870 -- 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

[inlong-website] branch master updated: [INLONG-870][Sort] Update starrocks.md for correct use of streamload

2023-10-08 Thread zirui
This is an automated email from the ASF dual-hosted git repository. zirui pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/inlong-website.git The following commit(s) were added to refs/heads/master by this push: new 13af8de342 [INLONG-870][Sort] Update star

Re: [I] [Improve] update how-to-verify doc to use correct test command [inlong-website]

2023-10-08 Thread via GitHub
dockerzhang closed issue #873: [Improve] update how-to-verify doc to use correct test command URL: https://github.com/apache/inlong-website/issues/873 -- 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

Re: [PR] [INLONG-873][Release] Update how-to-verify doc to use correct test command [inlong-website]

2023-10-08 Thread via GitHub
dockerzhang merged PR #874: URL: https://github.com/apache/inlong-website/pull/874 -- 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: commits-unsubscr...@i

[inlong-website] branch master updated: [INLONG-873][Release] Update how-to-verify doc to use correct test command (#874)

2023-10-08 Thread dockerzhang
This is an automated email from the ASF dual-hosted git repository. dockerzhang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/inlong-website.git The following commit(s) were added to refs/heads/master by this push: new c525a1dcb9 [INLONG-873][Release] Up

Re: [PR] [INLONG-8999][Sort] Add Sqlserver connector on flink 1.15 [inlong]

2023-10-08 Thread via GitHub
EMsnap commented on code in PR #9008: URL: https://github.com/apache/inlong/pull/9008#discussion_r1349680480 ## inlong-sort/sort-end-to-end-tests/sort-end-to-end-tests-v1.15/src/test/java/org/apache/inlong/sort/tests/utils/MSSQLServerContainer.java: ## @@ -0,0 +1,148 @@ +/* + *

Re: [I] [Improve][Doc] Pick #869 and #863 to 1.9.0 [inlong-website]

2023-10-08 Thread via GitHub
dockerzhang closed issue #875: [Improve][Doc] Pick #869 and #863 to 1.9.0 URL: https://github.com/apache/inlong-website/issues/875 -- 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

Re: [PR] [INLONG-875][Doc] Pick #869 and #863 to 1.9.0 [inlong-website]

2023-10-08 Thread via GitHub
dockerzhang merged PR #876: URL: https://github.com/apache/inlong-website/pull/876 -- 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: commits-unsubscr...@i

[inlong-website] branch master updated: [INLONG-875][Doc] Pick #869 and #863 to 1.9.0 (#876)

2023-10-08 Thread dockerzhang
This is an automated email from the ASF dual-hosted git repository. dockerzhang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/inlong-website.git The following commit(s) were added to refs/heads/master by this push: new 209207de7c [INLONG-875][Doc] Pick #

[I] [Improve] Update document of TubeMQ extract node [inlong-website]

2023-10-08 Thread via GitHub
fancycoderzf opened a new issue, #877: URL: https://github.com/apache/inlong-website/issues/877 ### Description Update document of TubeMQ extract node ### Are you willing to submit PR? - [X] Yes, I am willing to submit a PR! ### Code of Conduct - [X] I agree

[I] [Improve] Update document of HBase load node [inlong-website]

2023-10-08 Thread via GitHub
fancycoderzf opened a new issue, #878: URL: https://github.com/apache/inlong-website/issues/878 ### Description Update document of HBase load node ### Are you willing to submit PR? - [X] Yes, I am willing to submit a PR! ### Code of Conduct - [X] I agree to

[PR] Update document of TubeMQ extract node [inlong-website]

2023-10-08 Thread via GitHub
fancycoderzf opened a new pull request, #879: URL: https://github.com/apache/inlong-website/pull/879 ### Prepare a Pull Request *(Change the title refer to the following example)* - Title Example: [INLONG-XYZ][Component] Title of the pull request *(The following *XYZ* should

Re: [PR] [INLONG-8961][Manager] Server-side request forgery attack prevention in some classes [inlong]

2023-10-08 Thread via GitHub
sunrisefromdark commented on code in PR #9012: URL: https://github.com/apache/inlong/pull/9012#discussion_r1349689699 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/ck/ClickHouseJdbcUtils.java: ## @@ -39,37 +39,153 @@ public clas

Re: [PR] [INLONG-8961][Manager] Server-side request forgery attack prevention in some classes [inlong]

2023-10-08 Thread via GitHub
sunrisefromdark commented on code in PR #9012: URL: https://github.com/apache/inlong/pull/9012#discussion_r1349689758 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/ck/ClickHouseJdbcUtils.java: ## @@ -39,37 +39,104 @@ public clas

[PR] [Feature][Sort] Add TubeMQ source and sink connector on flink 1.15 [inlong]

2023-10-08 Thread via GitHub
fancycoderzf opened a new pull request, #9031: URL: https://github.com/apache/inlong/pull/9031 ### Prepare a Pull Request - Fixes #8922 ### Motivation *Add TubeMQ source and sink connector on flink 1.15.* ### Modifications *Add TubeMQ source and sink connector

Re: [PR] [INLONG-8961][Manager] Server-side request forgery attack prevention in some classes [inlong]

2023-10-08 Thread via GitHub
sunrisefromdark commented on code in PR #9012: URL: https://github.com/apache/inlong/pull/9012#discussion_r1349711189 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/starrocks/StarRocksJdbcUtils.java: ## @@ -44,25 +45,50 @@ public

[PR] [INLONG-9009][Sort] Add HBase source and sink connector on flink 1.15 [inlong]

2023-10-08 Thread via GitHub
fancycoderzf opened a new pull request, #9032: URL: https://github.com/apache/inlong/pull/9032 ### Prepare a Pull Requestt - Fixes #9009 ### Motivation *Add HBase source and sink connector on flink 1.15* ### Modifications *Add HBase source and sink connector

Re: [I] [Improve][Security] The `size` option isn't honored after following a redirect in node-fetch [inlong]

2023-10-08 Thread via GitHub
github-actions[bot] commented on issue #8691: URL: https://github.com/apache/inlong/issues/8691#issuecomment-1752247206 This issue is stale because it has been open for 60 days with no activity. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [I] [Improve][Manager] Flink - Modify the calling method from SDK to HTTP [inlong]

2023-10-08 Thread via GitHub
github-actions[bot] commented on issue #8677: URL: https://github.com/apache/inlong/issues/8677#issuecomment-1752247240 This issue is stale because it has been open for 60 days with no activity. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [INLONG-8804][Dashboard] Separate plugin images and UI components [inlong]

2023-10-08 Thread via GitHub
leezng commented on code in PR #9006: URL: https://github.com/apache/inlong/pull/9006#discussion_r1349816487 ## inlong-dashboard/src/ui/components/CheckCard/index.tsx: ## @@ -62,13 +63,9 @@ const CheckCard: React.FC = ({ options, value, onChange, disable */ (async ()

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
fuweng11 commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349839557 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -55,6 +64,9 @@ public class ClsReso

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349844491 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -136,14 +168,33 @@ private ClsClien

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349844360 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -152,15 +203,86 @@ private void cre

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
fuweng11 commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349839871 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -78,32 +90,27 @@ public void createS

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349845136 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -136,14 +168,33 @@ private ClsClien

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
fuweng11 commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349840308 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -113,6 +120,31 @@ private void creat

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349845440 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -78,32 +90,27 @@ public void create

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349845920 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -55,6 +64,9 @@ public class ClsRes

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349845989 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/sink/cls/ClsResourceOperator.java: ## @@ -78,32 +90,27 @@ public void create

Re: [PR] [INLONG-8958][Manager] Create cls topic fail [inlong]

2023-10-08 Thread via GitHub
castorqin commented on code in PR #9029: URL: https://github.com/apache/inlong/pull/9029#discussion_r1349889748 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/sink/AbstractSinkOperator.java: ## @@ -218,14 +219,19 @@ public void deleteOpt(Stream

Re: [PR] [INLONG-8997][Sort] Keep the logic the same of mysql and sqlserver test container with flink v1.13 in end to end test [inlong]

2023-10-08 Thread via GitHub
EMsnap commented on PR #9021: URL: https://github.com/apache/inlong/pull/9021#issuecomment-1752398106 Please add a description on what problem the pr solves, btw why add a class that's not used ? -- This is an automated message from the Apache Git Service. To respond to the message, plea