Re: [PR] [WIP][Improve][Format-Json] Remove assert key word. [seatunnel]

2023-11-25 Thread via GitHub
CheneyYin commented on code in PR #5919: URL: https://github.com/apache/seatunnel/pull/5919#discussion_r1405337732 ## seatunnel-formats/seatunnel-format-json/src/main/java/org/apache/seatunnel/format/json/canal/CanalJsonDeserializationSchema.java: ## @@ -113,92 +113,97 @@ public

Re: [PR] [WIP][Improve][Format-Json] Remove assert key word. [seatunnel]

2023-11-25 Thread via GitHub
CheneyYin commented on code in PR #5919: URL: https://github.com/apache/seatunnel/pull/5919#discussion_r1405337732 ## seatunnel-formats/seatunnel-format-json/src/main/java/org/apache/seatunnel/format/json/canal/CanalJsonDeserializationSchema.java: ## @@ -113,92 +113,97 @@ public

[PR] [Improve] fix empty line in config/plugin_config causes BUILD FAILURE [seatunnel]

2023-11-25 Thread via GitHub
chaos-cn opened a new pull request, #5921: URL: https://github.com/apache/seatunnel/pull/5921 If config/plugin_config has empty line mvn dependency:get -DgroupId=org.apache.seatunnel -DartifactId= -Dversion=2.3.3 -Ddest=/opt/apache-seatunnel-2.3.3/connectors BUILD FAILURE

Re: [PR] [Improve] fix empty line in config/plugin_config causes BUILD FAILURE [seatunnel]

2023-11-25 Thread via GitHub
chaos-cn closed pull request #5920: [Improve] fix empty line in config/plugin_config causes BUILD FAILURE URL: https://github.com/apache/seatunnel/pull/5920 -- 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

Re: [PR] [Improve][StarRocksSink] add http socket timeout. [seatunnel]

2023-11-25 Thread via GitHub
Hisoka-X commented on code in PR #5918: URL: https://github.com/apache/seatunnel/pull/5918#discussion_r1405290776 ## seatunnel-connectors-v2/connector-starrocks/src/main/java/org/apache/seatunnel/connectors/seatunnel/starrocks/config/SinkConfig.java: ## @@ -90,6 +92,8 @@ public

Re: [PR] [WIP][Improve][Format-Json] Remove assert key word. [seatunnel]

2023-11-25 Thread via GitHub
Hisoka-X commented on code in PR #5919: URL: https://github.com/apache/seatunnel/pull/5919#discussion_r1405285485 ## seatunnel-formats/seatunnel-format-json/src/main/java/org/apache/seatunnel/format/json/canal/CanalJsonDeserializationSchema.java: ## @@ -113,92 +113,97 @@ public

Re: [I] [Bug] [Module Name] Bug title [seatunnel]

2023-11-25 Thread via GitHub
github-actions[bot] commented on issue #5713: URL: https://github.com/apache/seatunnel/issues/5713#issuecomment-1826451668 This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.

Re: [I] [Bug] [Connector-mongodb] The mongodb connector causes a consistent number of connections in the business system not to be released. Task run completed [seatunnel]

2023-11-25 Thread via GitHub
github-actions[bot] commented on issue #5715: URL: https://github.com/apache/seatunnel/issues/5715#issuecomment-1826451660 This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.

Re: [I] [Feature][Connector-V2] Support multi-table sink feature for hive [seatunnel]

2023-11-25 Thread via GitHub
github-actions[bot] commented on issue #5717: URL: https://github.com/apache/seatunnel/issues/5717#issuecomment-1826451647 This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.

Re: [I] [Feature][Connector-V2] Support multi-table sink feature for kudu [seatunnel]

2023-11-25 Thread via GitHub
github-actions[bot] commented on issue #5719: URL: https://github.com/apache/seatunnel/issues/5719#issuecomment-1826451635 This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.

Re: [I] [Feature][Connector-V2] Support multi-table sink feature for doris [seatunnel]

2023-11-25 Thread via GitHub
github-actions[bot] commented on issue #5718: URL: https://github.com/apache/seatunnel/issues/5718#issuecomment-1826451641 This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.

[PR] fix empty line in config/plugin_config causes BUILD FAILURE [seatunnel]

2023-11-25 Thread via GitHub
chaos-cn opened a new pull request, #5920: URL: https://github.com/apache/seatunnel/pull/5920 if config/plugin_config has empty line mvn dependency:get -DgroupId=org.apache.seatunnel -DartifactId= -Dversion=2.3.3 -Ddest=/opt/apache-seatunnel-2.3.3/connectors BUILD FAILURE

Re: [PR] [Feature][Zeta] In batch mode, checkpoint can be disabled [seatunnel]

2023-11-25 Thread via GitHub
hailin0 commented on PR #5914: URL: https://github.com/apache/seatunnel/pull/5914#issuecomment-1826349118 > update all docs & e2e example for batch task @happyboy1024 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [Feature][Zeta] In batch mode, checkpoint can be disabled [seatunnel]

2023-11-25 Thread via GitHub
hailin0 commented on code in PR #5914: URL: https://github.com/apache/seatunnel/pull/5914#discussion_r1404882189 ## seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/master/JobMaster.java: ## @@ -274,6 +275,11 @@ private CheckpointConfig c

Re: [PR] [Bugfix][Zeta] Fix the checkpoint timeout (#5722) [seatunnel]

2023-11-25 Thread via GitHub
hailin0 commented on code in PR #5904: URL: https://github.com/apache/seatunnel/pull/5904#discussion_r1404878883 ## seatunnel-connectors-v2/connector-file/connector-file-base/src/main/java/org/apache/seatunnel/connectors/seatunnel/file/sink/writer/JsonWriteStrategy.java: ## @@ -

Re: [PR] [Bugfix][Zeta] Fix the checkpoint timeout (#5722) [seatunnel]

2023-11-25 Thread via GitHub
hailin0 commented on PR #5904: URL: https://github.com/apache/seatunnel/pull/5904#issuecomment-1826301727 > Similar to [5722](https://github.com/apache/seatunnel/issues/5722), when I use ES's scroll query or read large files, the checkpoint timeout problem will also occur. > I found that

Re: [PR] [Bugfix][Zeta] Fix the checkpoint timeout (#5722) [seatunnel]

2023-11-25 Thread via GitHub
hailin0 commented on code in PR #5904: URL: https://github.com/apache/seatunnel/pull/5904#discussion_r1404875293 ## seatunnel-connectors-v2/connector-elasticsearch/src/main/java/org/apache/seatunnel/connectors/seatunnel/elasticsearch/source/ElasticsearchSourceReader.java: ## @@

Re: [PR] [Bugfix][Zeta] Fix the checkpoint timeout (#5722) [seatunnel]

2023-11-25 Thread via GitHub
hailin0 commented on code in PR #5904: URL: https://github.com/apache/seatunnel/pull/5904#discussion_r1404874140 ## seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/source/JdbcSourceReader.java: ## @@ -60,25 +60,23 @@ public voi

Re: [PR] [WIP][Improve][Format-Json] Remove assert key word. [seatunnel]

2023-11-25 Thread via GitHub
CheneyYin commented on code in PR #5919: URL: https://github.com/apache/seatunnel/pull/5919#discussion_r1404864652 ## seatunnel-formats/seatunnel-format-json/src/main/java/org/apache/seatunnel/format/json/canal/CanalJsonDeserializationSchema.java: ## @@ -113,92 +113,97 @@ public

Re: [PR] [Feature][Zeta] In batch mode, checkpoint can be disabled [seatunnel]

2023-11-25 Thread via GitHub
happyboy1024 commented on code in PR #5914: URL: https://github.com/apache/seatunnel/pull/5914#discussion_r1404847571 ## seatunnel-e2e/seatunnel-engine-e2e/connector-seatunnel-e2e-base/src/test/java/org/apache/seatunnel/engine/e2e/CheckpointEnableIT.java: ## @@ -0,0 +1,136 @@ +/

(seatunnel) branch dev updated: [Improve]Change System.out.println to log output. (#5912)

2023-11-25 Thread fanjia
This is an automated email from the ASF dual-hosted git repository. fanjia pushed a commit to branch dev in repository https://gitbox.apache.org/repos/asf/seatunnel.git The following commit(s) were added to refs/heads/dev by this push: new bbedb07a9c [Improve]Change System.out.println to lo

Re: [PR] [Improve]Change System.out.println to log output. [seatunnel]

2023-11-25 Thread via GitHub
Hisoka-X merged PR #5912: URL: https://github.com/apache/seatunnel/pull/5912 -- 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...@seatunn

Re: [I] [Bug] [Connector-v2][connector-common] Is `assert` used to control program flow? [seatunnel]

2023-11-25 Thread via GitHub
hailin0 closed issue #5902: [Bug] [Connector-v2][connector-common] Is `assert` used to control program flow? URL: https://github.com/apache/seatunnel/issues/5902 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

(seatunnel) branch dev updated: [Improve][Connector-V2][Common] Remove assert key word. (#5915)

2023-11-25 Thread wanghailin
This is an automated email from the ASF dual-hosted git repository. wanghailin pushed a commit to branch dev in repository https://gitbox.apache.org/repos/asf/seatunnel.git The following commit(s) were added to refs/heads/dev by this push: new d757dcd1fc [Improve][Connector-V2][Common] Remo

Re: [PR] [Improve][Connector-V2][Common] Remove assert key word. [seatunnel]

2023-11-25 Thread via GitHub
hailin0 merged PR #5915: URL: https://github.com/apache/seatunnel/pull/5915 -- 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...@seatunne

[PR] [Improve][Format-Json] Remove assert key word. [seatunnel]

2023-11-25 Thread via GitHub
CheneyYin opened a new pull request, #5919: URL: https://github.com/apache/seatunnel/pull/5919 ### Purpose of this pull request Remove assert key word. Related: - #5902 - #5915 ### Does this PR introduce _any_ user-facing change? ### How was

[PR] [Improve][StarRocksSink] add http socket timeout. [seatunnel]

2023-11-25 Thread via GitHub
lightzhao opened a new pull request, #5918: URL: https://github.com/apache/seatunnel/pull/5918 ### Purpose of this pull request ### Does this PR introduce _any_ user-facing change? ### How was this patch tested? ### Check list *

Re: [PR] [Docs][Connector-V2] Reconstruct the Socket Source and SftpFile connector document [seatunnel]

2023-11-25 Thread via GitHub
Hisoka-X commented on PR #5386: URL: https://github.com/apache/seatunnel/pull/5386#issuecomment-1826257329 > Please fix the conflict Done. Please take a look again. @zhilinli123 @hailin0 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [Docs][Connector-V2] Reconstruct the Socket Source and SftpFile connector document [seatunnel]

2023-11-25 Thread via GitHub
Hisoka-X commented on code in PR #5386: URL: https://github.com/apache/seatunnel/pull/5386#discussion_r1404806626 ## docs/en/connector-v2/source/Socket.md: ## @@ -15,58 +17,66 @@ Used to read data from Socket. - [ ] [parallelism](../../concept/connector-v2-features.md) - [ ] [

Re: [I] [Bug] [seatunnel-connectors-v2] when I use connector-cdc-mysql to read mysql-bin data,a ClassCastException error is reported [seatunnel]

2023-11-25 Thread via GitHub
fengcheche commented on issue #3929: URL: https://github.com/apache/seatunnel/issues/3929#issuecomment-1826255570 2.3.3还有这个错误 -- 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: [I] [Bug] [mysql cdc] can't read data from mysql cdc source [seatunnel]

2023-11-25 Thread via GitHub
fengcheche commented on issue #5831: URL: https://github.com/apache/seatunnel/issues/5831#issuecomment-1826254478 ![image](https://github.com/apache/seatunnel/assets/7406807/99602290-454b-475d-b55a-45947445b591) -- This is an automated message from the Apache Git Service. To respond t

Re: [I] [Bug] [mysql cdc] can't read data from mysql cdc source [seatunnel]

2023-11-25 Thread via GitHub
fengcheche commented on issue #5831: URL: https://github.com/apache/seatunnel/issues/5831#issuecomment-1826254325 不用web配置,手动输入切换成下面这样去启动还是报一样的错误 Without web configuration, manually enter to switch to the following to start still report the same error ``` env { execution.par

Re: [PR] [Feature][Zeta] In batch mode, checkpoint can be disabled [seatunnel]

2023-11-25 Thread via GitHub
Hisoka-X commented on code in PR #5914: URL: https://github.com/apache/seatunnel/pull/5914#discussion_r1404798203 ## seatunnel-e2e/seatunnel-engine-e2e/connector-seatunnel-e2e-base/src/test/java/org/apache/seatunnel/engine/e2e/CheckpointEnableIT.java: ## @@ -0,0 +1,136 @@ +/* +

Re: [PR] [Docs][Connector-V2][Oss]Reconstruct the OssFile connector document [seatunnel]

2023-11-25 Thread via GitHub
Hisoka-X commented on PR #5233: URL: https://github.com/apache/seatunnel/pull/5233#issuecomment-1826249067 > Pending conflict resolution Done. Please take a look again. Thanks @Carl-Zhou-CN @zhilinli123 @hailin0 -- This is an automated message from the Apache Git Service. To respon