[GitHub] [seatunnel] kim-up commented on pull request #5112: [Feature][zeta][monitoring] Support exposing monitoring metrics by prometheus exporter protocol (#5070)

2023-08-02 Thread via GitHub
kim-up commented on PR #5112: URL: https://github.com/apache/seatunnel/pull/5112#issuecomment-1663377422 > Can we add the [slowoperation](https://docs.hazelcast.com/imdg/4.2/performance/slowoperationdetector) metric? It's quite valuable for troubleshooting. You can refer to the Hazelcast M

[GitHub] [seatunnel] zhilinli123 opened a new pull request, #5209: [bug][jdbc][oracle]Fix the Oracle number type mapping problem

2023-08-02 Thread via GitHub
zhilinli123 opened a new pull request, #5209: URL: https://github.com/apache/seatunnel/pull/5209 https://github.com/apache/seatunnel/issues/5204 -- 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 th

[GitHub] [seatunnel] sunxiaojian opened a new pull request, #4516: [Feature][Connector-V2] connector-kafka source support data conversion extracted by kafka connect source

2023-08-02 Thread via GitHub
sunxiaojian opened a new pull request, #4516: URL: https://github.com/apache/seatunnel/pull/4516 ## Purpose of this pull request #4137 ## Check list * [ ] Code changed are covered with tests, or it does not need tests for reason: * [ ] If any new Jar binar

[GitHub] [seatunnel] hailin0 closed pull request #4516: [Feature][Connector-V2] connector-kafka source support data conversion extracted by kafka connect source

2023-08-02 Thread via GitHub
hailin0 closed pull request #4516: [Feature][Connector-V2] connector-kafka source support data conversion extracted by kafka connect source URL: https://github.com/apache/seatunnel/pull/4516 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [seatunnel] liugddx closed pull request #5208: [CI] Split updated modules integration test for part 5

2023-08-02 Thread via GitHub
liugddx closed pull request #5208: [CI] Split updated modules integration test for part 5 URL: https://github.com/apache/seatunnel/pull/5208 -- 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 speci

[GitHub] [seatunnel] hailin0 commented on pull request #4713: [draft][Spark streaming] fix spark streaming

2023-08-02 Thread via GitHub
hailin0 commented on PR #4713: URL: https://github.com/apache/seatunnel/pull/4713#issuecomment-1663327258 please check code style -- 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 com

[GitHub] [seatunnel] liugddx opened a new pull request, #5208: [CI] Split updated modules integration test for part 5

2023-08-02 Thread via GitHub
liugddx opened a new pull request, #5208: URL: https://github.com/apache/seatunnel/pull/5208 ## Purpose of this pull request ## Check list * [ ] Code changed are covered with tests, or it does not need tests for reason: * [ ] If any new Jar binary package add

[GitHub] [seatunnel-web] dependabot[bot] opened a new pull request, #95: Bump vite from 4.1.4 to 4.1.5 in /seatunnel-ui

2023-08-02 Thread via GitHub
dependabot[bot] opened a new pull request, #95: URL: https://github.com/apache/seatunnel-web/pull/95 Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 4.1.4 to 4.1.5. Changelog Sourced from https://github.com/vitejs/vite/blob/v4.1.5/packages/vite/CHANGELOG.m

[seatunnel-web] branch dependabot/npm_and_yarn/seatunnel-ui/vite-4.1.5 created (now 708a5d71)

2023-08-02 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/seatunnel-ui/vite-4.1.5 in repository https://gitbox.apache.org/repos/asf/seatunnel-web.git at 708a5d71 Bump vite from 4.1.4 to 4.1.5 in /seatunnel-ui No new r

[GitHub] [seatunnel] XiaoJiang521 opened a new pull request, #5207: [bugfix][DM] dm database, schema is database,so table no need schema

2023-08-02 Thread via GitHub
XiaoJiang521 opened a new pull request, #5207: URL: https://github.com/apache/seatunnel/pull/5207 ## Purpose of this pull request ## Check list * [ ] Code changed are covered with tests, or it does not need tests for reason: * [ ] If any new Jar binary packag

[GitHub] [seatunnel] Hisoka-X commented on pull request #5114: [Feature][Zeta] Support history service record job execute error

2023-08-02 Thread via GitHub
Hisoka-X commented on PR #5114: URL: https://github.com/apache/seatunnel/pull/5114#issuecomment-1663210811 > I think adding a new state should solve this problem in a new PR rather than in the current PR Yep, we should do it in another PR. -- This is an automated message from the A

[GitHub] [seatunnel] junmingliu commented on issue #5204: [Bug] [JDBC] Oracle source The current field is DecimalType containing decimals: 18 Unable to support

2023-08-02 Thread via GitHub
junmingliu commented on issue #5204: URL: https://github.com/apache/seatunnel/issues/5204#issuecomment-1663199784 > I understand that it's a positive integer,when scale is zero,there are two judgments。 when basicType is long,the range of long value is between -9223372036854775808

[GitHub] [seatunnel] zhibinF commented on a diff in pull request #5107: [Feature][Zeta][REST-API]Add REST API To Submit Job

2023-08-02 Thread via GitHub
zhibinF commented on code in PR #5107: URL: https://github.com/apache/seatunnel/pull/5107#discussion_r1282557314 ## seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/job/JobImmutableInformationEnv.java: ## Review Comment: Yes, the mai

[GitHub] [seatunnel] github-actions[bot] commented on issue #4974: [Bug] [JDBC] SqlServer to oracle literal does not match format string

2023-08-02 Thread via GitHub
github-actions[bot] commented on issue #4974: URL: https://github.com/apache/seatunnel/issues/4974#issuecomment-1663128558 This issue has been closed because it has not received response for too long time. You could reopen it if you encountered similar problems in the future. -- This is a

[GitHub] [seatunnel] github-actions[bot] closed issue #4974: [Bug] [JDBC] SqlServer to oracle literal does not match format string

2023-08-02 Thread via GitHub
github-actions[bot] closed issue #4974: [Bug] [JDBC] SqlServer to oracle literal does not match format string URL: https://github.com/apache/seatunnel/issues/4974 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [seatunnel] github-actions[bot] commented on issue #5010: [Bug] [Mysql-cdc] send Mysql data to kafka

2023-08-02 Thread via GitHub
github-actions[bot] commented on issue #5010: URL: https://github.com/apache/seatunnel/issues/5010#issuecomment-1663128516 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.

[GitHub] [seatunnel] junmingliu opened a new issue, #5204: [Bug] [JDBC] Oracle source The current field is DecimalType containing decimals: 18 Unable to support

2023-08-02 Thread via GitHub
junmingliu opened a new issue, #5204: URL: https://github.com/apache/seatunnel/issues/5204 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/seatunnel/issues?q=is%3Aissue+label%3A%22bug%22) and found no similar issues. ### What happene

[seatunnel] branch dev updated: [e2e] kafka e2e error (#5200)

2023-08-02 Thread liugddx
This is an automated email from the ASF dual-hosted git repository. liugddx 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 00e0ce9e2 [e2e] kafka e2e error (#5200) 00e0ce9e2

[GitHub] [seatunnel] liugddx merged pull request #5200: [e2e][bugfix] kafka e2e error

2023-08-02 Thread via GitHub
liugddx merged PR #5200: URL: https://github.com/apache/seatunnel/pull/5200 -- 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

[GitHub] [seatunnel] SharkAndCat opened a new issue, #5203: the file name of sink

2023-08-02 Thread via GitHub
SharkAndCat opened a new issue, #5203: URL: https://github.com/apache/seatunnel/issues/5203 ### Search before asking - [X] I had searched in the [feature](https://github.com/apache/seatunnel/issues?q=is%3Aissue+label%3A%22Feature%22) and found no similar feature requirement.

[GitHub] [seatunnel] ic4y commented on pull request #5101: [Doc] Improve S3File Source & S3File Sink document

2023-08-02 Thread via GitHub
ic4y commented on PR #5101: URL: https://github.com/apache/seatunnel/pull/5101#issuecomment-1661947470 @EricJoy2048 Conflicts need to be resolved. -- 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

[GitHub] [seatunnel] ic4y commented on a diff in pull request #5107: [Feature][Zeta][REST-API]Add REST API To Submit Job

2023-08-02 Thread via GitHub
ic4y commented on code in PR #5107: URL: https://github.com/apache/seatunnel/pull/5107#discussion_r1281703025 ## seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/job/JobImmutableInformationEnv.java: ## Review Comment: This class has

[GitHub] [seatunnel] ic4y commented on pull request #5112: [Feature][zeta][monitoring] Support exposing monitoring metrics by prometheus exporter protocol (#5070)

2023-08-02 Thread via GitHub
ic4y commented on PR #5112: URL: https://github.com/apache/seatunnel/pull/5112#issuecomment-1661926829 Can we add the [slowoperation](https://docs.hazelcast.com/imdg/4.2/performance/slowoperationdetector) metric? It's quite valuable for troubleshooting. You can refer to the Hazelcast Manag

[GitHub] [seatunnel] lightzhao commented on pull request #4871: [Docs][Connector-V2][HDFS]Refactor connector-v2 docs using unified format HDFS.

2023-08-02 Thread via GitHub
lightzhao commented on PR #4871: URL: https://github.com/apache/seatunnel/pull/4871#issuecomment-1661866790 @EricJoy2048 @zhilinli123 PTAL. -- 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 spe

[GitHub] [seatunnel] Lifu12 opened a new issue, #5202: [Feature][Connector v2] add opengaussdb Module

2023-08-02 Thread via GitHub
Lifu12 opened a new issue, #5202: URL: https://github.com/apache/seatunnel/issues/5202 ### Search before asking - [X] I had searched in the [feature](https://github.com/apache/seatunnel/issues?q=is%3Aissue+label%3A%22Feature%22) and found no similar feature requirement. ##

[GitHub] [seatunnel] lightzhao commented on pull request #4975: [Feature][Connector-v2][RedisSink]Support redis to set expiration time.

2023-08-02 Thread via GitHub
lightzhao commented on PR #4975: URL: https://github.com/apache/seatunnel/pull/4975#issuecomment-1661859402 @EricJoy2048 @TyrantLucifer @liugddx @TaoZex @Hisoka-X PTAL. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [seatunnel] kim-up closed issue #5198: [Zeta] [savepoint] Job resume from savepoint failed

2023-08-02 Thread via GitHub
kim-up closed issue #5198: [Zeta] [savepoint] Job resume from savepoint failed URL: https://github.com/apache/seatunnel/issues/5198 -- 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] [seatunnel] zhilinli123 commented on pull request #5201: [Improve][e2e]The e2e test case allows cleaning the image file after completion

2023-08-02 Thread via GitHub
zhilinli123 commented on PR #5201: URL: https://github.com/apache/seatunnel/pull/5201#issuecomment-1661844986 > Does testcontainers provide an API for deleting images? It seems that only the methods for creating and closing containers are provided, and the methods for deleting images

[GitHub] [seatunnel] ic4y commented on pull request #5201: [Improve][e2e]The e2e test case allows cleaning the image file after completion

2023-08-02 Thread via GitHub
ic4y commented on PR #5201: URL: https://github.com/apache/seatunnel/pull/5201#issuecomment-1661835293 Does testcontainers provide an API for deleting images? -- 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: [Bug] [connector-v2] PostgreSQL versions below 9.5 are compatible use cdc sync problem (#5120)

2023-08-02 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 9af696a1d [Bug] [connector-v2] PostgreSQL versions

[GitHub] [seatunnel] Hisoka-X merged pull request #5120: [Bug] [connector-v2] PostgreSQL versions below 9.5 are compatible use cdc sync problem

2023-08-02 Thread via GitHub
Hisoka-X merged PR #5120: URL: https://github.com/apache/seatunnel/pull/5120 -- 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

[GitHub] [seatunnel] zhilinli123 opened a new pull request, #5201: [Improve][e2e]Clean up allowed after docker image

2023-08-02 Thread via GitHub
zhilinli123 opened a new pull request, #5201: URL: https://github.com/apache/seatunnel/pull/5201 (no comment) -- 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

[GitHub] [seatunnel] ic4y commented on pull request #5114: [Feature][Zeta] Support history service record job execute error

2023-08-02 Thread via GitHub
ic4y commented on PR #5114: URL: https://github.com/apache/seatunnel/pull/5114#issuecomment-1661772122 I suggest adding a new JobStatus, UNKNOWN, when jobState is null. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [seatunnel] ic4y commented on pull request #5114: [Feature][Zeta] Support history service record job execute error

2023-08-02 Thread via GitHub
ic4y commented on PR #5114: URL: https://github.com/apache/seatunnel/pull/5114#issuecomment-1661765219 Relevant PR: https://github.com/apache/seatunnel/pull/5007 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[seatunnel] branch dev updated: [Bug][Improve][LocalFileSink]Fix LocalFile Sink file_format_type. (#5118)

2023-08-02 Thread ic4y
This is an automated email from the ASF dual-hosted git repository. ic4y 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 e717d80166 [Bug][Improve][LocalFileSink]Fix LocalFile

[GitHub] [seatunnel] ic4y merged pull request #5118: [Bug][Improve][LocalFileSink]Fix LocalFile Sink file_format_type.

2023-08-02 Thread via GitHub
ic4y merged PR #5118: URL: https://github.com/apache/seatunnel/pull/5118 -- 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...@seatunnel.a

[GitHub] [seatunnel] XiaoJiang521 opened a new pull request, #5200: [e2e][bugfix] kafka e2e error

2023-08-02 Thread via GitHub
XiaoJiang521 opened a new pull request, #5200: URL: https://github.com/apache/seatunnel/pull/5200 ## Purpose of this pull request ## Check list * [ ] Code changed are covered with tests, or it does not need tests for reason: * [ ] If any new Jar binary packag

[GitHub] [seatunnel] ic4y opened a new pull request, #5199: [bugfix] [savepoint test] Turn on the testSavepoint test.

2023-08-02 Thread via GitHub
ic4y opened a new pull request, #5199: URL: https://github.com/apache/seatunnel/pull/5199 ## Purpose of this pull request ## Check list * [ ] Code changed are covered with tests, or it does not need tests for reason: * [ ] If any new Jar binary package adding

[GitHub] [seatunnel] kim-up opened a new issue, #5198: [Zeta] [savepoint] Job resume from savepoint failed

2023-08-02 Thread via GitHub
kim-up opened a new issue, #5198: URL: https://github.com/apache/seatunnel/issues/5198 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/seatunnel/issues?q=is%3Aissue+label%3A%22bug%22) and found no similar issues. ### What happened

[GitHub] [seatunnel] ic4y commented on pull request #5135: [Feature][SeaTunnel Engine Checkpoint Storage] Checkpoint storage supports kafka compact topic

2023-08-02 Thread via GitHub
ic4y commented on PR #5135: URL: https://github.com/apache/seatunnel/pull/5135#issuecomment-1661691440 It would be best to add an e2e test, similar to org/apache/seatunnel/engine/e2e/ClusterFaultToleranceIT.java and org.apache.seatunnel.engine.server.checkpoint.SavePointTest. -- This is