[GitHub] [inlong] yunqingmoswu commented on a diff in pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
yunqingmoswu commented on code in PR #6541: URL: https://github.com/apache/inlong/pull/6541#discussion_r1024873064 ## inlong-sort/sort-connectors/doris/src/main/java/org/apache/inlong/sort/doris/table/DorisDynamicTableSink.java: ## @@ -86,30 +84,30 @@ public ChangelogMode getCha

[GitHub] [inlong] Yizhou-Yang commented on a diff in pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
Yizhou-Yang commented on code in PR #6541: URL: https://github.com/apache/inlong/pull/6541#discussion_r1024880532 ## inlong-sort/sort-connectors/doris/src/main/java/org/apache/inlong/sort/doris/table/DorisDynamicSchemaOutputFormat.java: ## @@ -198,15 +320,92 @@ private boolean c

[GitHub] [inlong] Yizhou-Yang commented on a diff in pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
Yizhou-Yang commented on code in PR #6541: URL: https://github.com/apache/inlong/pull/6541#discussion_r1024880764 ## inlong-sort/sort-connectors/doris/src/main/java/org/apache/inlong/sort/doris/table/DorisDynamicSchemaOutputFormat.java: ## @@ -198,15 +320,92 @@ private boolean c

[GitHub] [inlong] EMsnap commented on a diff in pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
EMsnap commented on code in PR #6541: URL: https://github.com/apache/inlong/pull/6541#discussion_r1024884480 ## inlong-sort/sort-connectors/doris/src/main/java/org/apache/inlong/sort/doris/table/DorisDynamicSchemaOutputFormat.java: ## @@ -493,8 +690,12 @@ public static class Bui

[GitHub] [inlong] EMsnap commented on pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
EMsnap commented on PR #6541: URL: https://github.com/apache/inlong/pull/6541#issuecomment-1318274165 Personally I think DorisDynamicSchemaOutputFormat.java contains too much code and some of them should be moved outside of itself -- This is an automated message from the Apache Git Servic

[GitHub] [inlong] Yizhou-Yang commented on a diff in pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
Yizhou-Yang commented on code in PR #6541: URL: https://github.com/apache/inlong/pull/6541#discussion_r1024930497 ## inlong-sort/sort-connectors/doris/src/main/java/org/apache/inlong/sort/doris/table/DorisDynamicTableSink.java: ## @@ -86,30 +84,30 @@ public ChangelogMode getChan

[GitHub] [inlong-website] dockerzhang closed issue #606: [Improve][Doc] Add an entrance guide for Standard and Lightweight Architecture

2022-11-17 Thread GitBox
dockerzhang closed issue #606: [Improve][Doc] Add an entrance guide for Standard and Lightweight Architecture URL: https://github.com/apache/inlong-website/issues/606 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [inlong-website] dockerzhang merged pull request #607: [INLONG-606][Doc] Add an entrance guide for Standard and Lightweight Architecture

2022-11-17 Thread GitBox
dockerzhang merged PR #607: URL: https://github.com/apache/inlong-website/pull/607 -- 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-606][Doc] Add an entrance guide for Standard and Lightweight Architecture (#607)

2022-11-17 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 4cd38be91b [INLONG-606][Doc] Add an

[GitHub] [inlong] EMsnap commented on a diff in pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
EMsnap commented on code in PR #6541: URL: https://github.com/apache/inlong/pull/6541#discussion_r1024936947 ## inlong-sort/sort-connectors/doris/src/main/java/org/apache/inlong/sort/doris/table/DorisDynamicSchemaOutputFormat.java: ## @@ -205,8 +327,76 @@ public synchronized voi

[GitHub] [inlong-website] dockerzhang merged pull request #608: [INLONG-589] Create the 1.4.0 version document

2022-11-17 Thread GitBox
dockerzhang merged PR #608: URL: https://github.com/apache/inlong-website/pull/608 -- 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

[GitHub] [inlong-website] dockerzhang closed issue #589: [Release] Create the 1.4.0 version document

2022-11-17 Thread GitBox
dockerzhang closed issue #589: [Release] Create the 1.4.0 version document URL: https://github.com/apache/inlong-website/issues/589 -- 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 commen

[GitHub] [inlong-website] dockerzhang opened a new issue, #609: [Improve][Doc] Connect all Sort-related document guides

2022-11-17 Thread GitBox
dockerzhang opened a new issue, #609: URL: https://github.com/apache/inlong-website/issues/609 ### Description Connect all Sort-related document guides. ### Are you willing to submit PR? - [X] Yes, I am willing to submit a PR! ### Code of Conduct - [X] I agr

[inlong-website] branch fix-609 created (now 68dd1356fc)

2022-11-17 Thread dockerzhang
This is an automated email from the ASF dual-hosted git repository. dockerzhang pushed a change to branch fix-609 in repository https://gitbox.apache.org/repos/asf/inlong-website.git at 68dd1356fc [INLONG-589] Create the 1.4.0 version document (#608) No new revisions were added by this up

[inlong-website] branch fix-609 updated: [INLONG-609][Doc] Connect all Sort-related document guides

2022-11-17 Thread dockerzhang
This is an automated email from the ASF dual-hosted git repository. dockerzhang pushed a commit to branch fix-609 in repository https://gitbox.apache.org/repos/asf/inlong-website.git The following commit(s) were added to refs/heads/fix-609 by this push: new 83e40e49fe [INLONG-609][Doc] Conn

[GitHub] [inlong-website] dockerzhang opened a new pull request, #610: [INLONG-609][Doc] Connect all Sort-related document guides

2022-11-17 Thread GitBox
dockerzhang opened a new pull request, #610: URL: https://github.com/apache/inlong-website/pull/610 ### 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

[GitHub] [inlong] dockerzhang merged pull request #6523: [INLONG-6522][Docker] Add notes for building arm images

2022-11-17 Thread GitBox
dockerzhang merged PR #6523: URL: https://github.com/apache/inlong/pull/6523 -- 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-6522][Docker] Add notes for building arm images (#6523)

2022-11-17 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 081c088f7 [INLONG-6522][Docker] Add notes f

[inlong] branch master updated: [INLONG-6572][Dashboard] Stream is not deleted by default when the sink is deleted (#6573)

2022-11-17 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 b1c7cdbba [INLONG-6572][Dashboard] Stream i

[GitHub] [inlong] dockerzhang merged pull request #6573: [INLONG-6572][Dashboard] Stream is not deleted by default when the sink is deleted

2022-11-17 Thread GitBox
dockerzhang merged PR #6573: URL: https://github.com/apache/inlong/pull/6573 -- 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-6568][Manager] Fix the problem that the status of the deleted source was incorrectly modified (#6569)

2022-11-17 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 01ec14730 [INLONG-6568][Manager] Fix the pr

[GitHub] [inlong] dockerzhang merged pull request #6569: [INLONG-6568][Manager] Not modify the status of the deleted source

2022-11-17 Thread GitBox
dockerzhang merged PR #6569: URL: https://github.com/apache/inlong/pull/6569 -- 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.

[GitHub] [inlong] dockerzhang merged pull request #6566: [INLONG-6565][Dashboard] The operation of refreshing the sink configuration needs to be executed when the group configuration is successful

2022-11-17 Thread GitBox
dockerzhang merged PR #6566: URL: https://github.com/apache/inlong/pull/6566 -- 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-6565][Dashboard] The operation of refreshing the sink configuration needs to be executed when the group configuration is successful (#6566)

2022-11-17 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 b64e7d8f4 [INLONG-6565][Dashboard] The oper

[GitHub] [inlong] dockerzhang merged pull request #6562: [INLONG-6561][Manager] Fix the InlongGroup cannot be deleted

2022-11-17 Thread GitBox
dockerzhang merged PR #6562: URL: https://github.com/apache/inlong/pull/6562 -- 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-6561][Manager] Fix the InlongGroup cannot be deleted (#6562)

2022-11-17 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 0bee46773 [INLONG-6561][Manager] Fix the In

[inlong] branch master updated: [INLONG-6563][Manager] It should throw error info when deleting an inlong group failed (#6564)

2022-11-17 Thread pacinogong
This is an automated email from the ASF dual-hosted git repository. pacinogong 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 22e90d143 [INLONG-6563][Manager] It should t

[GitHub] [inlong] gong merged pull request #6564: [INLONG-6563][Manager] It should throw error info when deleting an inlong group failed

2022-11-17 Thread GitBox
gong merged PR #6564: URL: https://github.com/apache/inlong/pull/6564 -- 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.apache.

[GitHub] [inlong-website] dockerzhang closed issue #609: [Improve][Doc] Connect all Sort-related document guides

2022-11-17 Thread GitBox
dockerzhang closed issue #609: [Improve][Doc] Connect all Sort-related document guides URL: https://github.com/apache/inlong-website/issues/609 -- 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 sp

[inlong-website] branch master updated: [INLONG-609][Doc] Connect all Sort-related document guides (#610)

2022-11-17 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 b62a12d866 [INLONG-609][Doc] Connec

[GitHub] [inlong-website] dockerzhang merged pull request #610: [INLONG-609][Doc] Connect all Sort-related document guides

2022-11-17 Thread GitBox
dockerzhang merged PR #610: URL: https://github.com/apache/inlong-website/pull/610 -- 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

[GitHub] [inlong] healchow commented on a diff in pull request #6571: [INLONG-6570][Manager] Ignore the length of the field when building sort config

2022-11-17 Thread GitBox
healchow commented on code in PR #6571: URL: https://github.com/apache/inlong/pull/6571#discussion_r1025895449 ## inlong-manager/manager-pojo/src/main/java/org/apache/inlong/manager/pojo/sort/util/FieldInfoUtils.java: ## @@ -172,7 +174,7 @@ public static FormatInfo convertFieldF

[GitHub] [inlong] yunqingmoswu commented on a diff in pull request #6541: [INLONG-6495][Sort] Support inlong metrics for single table doris and support metric restore for it

2022-11-17 Thread GitBox
yunqingmoswu commented on code in PR #6541: URL: https://github.com/apache/inlong/pull/6541#discussion_r1024954760 ## inlong-sort/sort-connectors/doris/src/main/java/org/apache/inlong/sort/doris/table/DorisDynamicSchemaOutputFormat.java: ## @@ -426,8 +614,10 @@ private boolean h