[GitHub] [inlong] fuweng11 opened a new pull request, #7402: [INLONG-7401][Manager] Fix the problem of partitions could not be created based on numPartitions in the inlong group

2023-02-21 Thread via GitHub
fuweng11 opened a new pull request, #7402: URL: https://github.com/apache/inlong/pull/7402 ### Prepare a Pull Request - Fixes #7401 ### Motivation Fix the problem of partitions could not be created based on numPartitions in the inlong group ### Modifications F

[GitHub] [inlong] e-mhui opened a new pull request, #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
e-mhui opened a new pull request, #7403: URL: https://github.com/apache/inlong/pull/7403 Upgrade Oracle CDC to version 2.3 - Fixes #7400 ### Motivation Upgrade Oracle CDC to version 2.3 ### Modifications 1. modify `pom.xml` -- This is an automated messag

[GitHub] [inlong] healchow commented on a diff in pull request #7402: [INLONG-7401][Manager] Fix the problem of partitions could not be created based on numPartitions

2023-02-21 Thread via GitHub
healchow commented on code in PR #7402: URL: https://github.com/apache/inlong/pull/7402#discussion_r1113189263 ## inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/queue/pulsar/PulsarOperator.java: ## @@ -49,7 +49,7 @@ public class PulsarOp

[GitHub] [inlong] healchow commented on pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
healchow commented on PR #7403: URL: https://github.com/apache/inlong/pull/7403#issuecomment-1438655116 Hi @e-mhui, I see you committed some unrelated commits in this PR. From my practice, you can reset hard from the master branch before pushing your commit. 1. `git stash -m "your

[GitHub] [inlong] healchow commented on pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
healchow commented on PR #7403: URL: https://github.com/apache/inlong/pull/7403#issuecomment-1438660739 In this scenario, you can do like this: 1. record your commit hash, it is `3f9f62ccd3ae421f90058ee0da12802f15327772`. 2. `git pull upstream master` 3. `git reset --hard upstrea

[GitHub] [inlong] e-mhui commented on pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
e-mhui commented on PR #7403: URL: https://github.com/apache/inlong/pull/7403#issuecomment-1439339017 > In this PR, it seems that you just need commit the final change, so you can do like this: > > ![image](https://user-images.githubusercontent.com/31994335/220387017-1d7e70fc-c963-40

[GitHub] [inlong] yunqingmoswu commented on a diff in pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
yunqingmoswu commented on code in PR #7403: URL: https://github.com/apache/inlong/pull/7403#discussion_r1113758310 ## inlong-sort/sort-connectors/oracle-cdc/pom.xml: ## @@ -30,7 +30,7 @@ Apache InLong - Sort-connector-oracle-cdc -1.7.0.Final +1.6.4.F

[GitHub] [inlong] EMsnap commented on pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
EMsnap commented on PR #7403: URL: https://github.com/apache/inlong/pull/7403#issuecomment-1439347149 Please add description on the reason for updating version -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [inlong] e-mhui commented on a diff in pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
e-mhui commented on code in PR #7403: URL: https://github.com/apache/inlong/pull/7403#discussion_r1113762226 ## inlong-sort/sort-connectors/oracle-cdc/pom.xml: ## @@ -30,7 +30,7 @@ Apache InLong - Sort-connector-oracle-cdc -1.7.0.Final +1.6.4.Final

[GitHub] [inlong] liaorui commented on a diff in pull request #7394: [INLONG-7391][Sort] Support CSV format and dirty data collecting for StarRocks connector all db migrating

2023-02-21 Thread via GitHub
liaorui commented on code in PR #7394: URL: https://github.com/apache/inlong/pull/7394#discussion_r1113763610 ## inlong-sort/sort-connectors/base/src/test/java/org/apache/inlong/sort/base/dirty/RegexReplaceTest.java: ## @@ -32,7 +32,7 @@ public void testRegexReplacement() throws

[GitHub] [inlong] e-mhui commented on pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
e-mhui commented on PR #7403: URL: https://github.com/apache/inlong/pull/7403#issuecomment-1439368431 > Please add description on the reason for updating version It has done. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [inlong] gong commented on pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
gong commented on PR #7403: URL: https://github.com/apache/inlong/pull/7403#issuecomment-1439382271 @e-mhui Please update license file of license module -- 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

[GitHub] [inlong] haifxu opened a new pull request, #7405: [INLONG-7404][Agent] Fix error of Redis connector

2023-02-21 Thread via GitHub
haifxu opened a new pull request, #7405: URL: https://github.com/apache/inlong/pull/7405 ### Prepare a Pull Request - Fixes #7404 ### Motivation 1. Missing redis job for Redis connector. ### Modifications 1. Add `RedisJob`. ### Verifying this change

[GitHub] [inlong] e-mhui commented on pull request #7403: [INLONG-7400][Sort] Upgrade Oracle CDC to version 2.3

2023-02-21 Thread via GitHub
e-mhui commented on PR #7403: URL: https://github.com/apache/inlong/pull/7403#issuecomment-1439386046 > @e-mhui Please update license file of license module done. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [inlong] healchow merged pull request #7402: [INLONG-7401][Manager] Fix that cannot create Pulsar partitions based on numPartitions param

2023-02-21 Thread via GitHub
healchow merged PR #7402: URL: https://github.com/apache/inlong/pull/7402 -- 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.apa

[inlong] branch master updated (730feee34 -> 2c6de2ac9)

2023-02-21 Thread healchow
This is an automated email from the ASF dual-hosted git repository. healchow pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/inlong.git from 730feee34 [INLONG-7395][Manager] Reduce log in SortSouceService and SortClusterService (#7396) add 2c6de2ac9 [