Re: [PR] [Improve][SeaTunnel-Web] Change JobStatus to enum type to avoid hard coding [seatunnel-web]

2024-09-07 Thread via GitHub
wuchunfu commented on PR #205: URL: https://github.com/apache/seatunnel-web/pull/205#issuecomment-2335108615 > @wuchunfu In your attached snapshot I can not see how many tests are run, how many failed or passed, can you please attach snapshot like below ![image](https://private-user-images.

(seatunnel-web) branch main updated: [Improve][SeaTunnel-Web] Change JobStatus to enum type to avoid hard coding (#205)

2024-09-07 Thread fanjia
This is an automated email from the ASF dual-hosted git repository. fanjia pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/seatunnel-web.git The following commit(s) were added to refs/heads/main by this push: new 94d8a394 [Improve][SeaTunnel-Web] Change JobS

[PR] [Feature][Seatunnel-web]Add support to configure placeholder with default value in the job config. [seatunnel-web]

2024-09-07 Thread via GitHub
arshadmohammad opened a new pull request, #208: URL: https://github.com/apache/seatunnel-web/pull/208 ## 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 pa

Re: [PR] [Feature][Seatunnel-web]Add support to configure placeholder with default value in the job config. [seatunnel-web]

2024-09-07 Thread via GitHub
arshadmohammad commented on PR #208: URL: https://github.com/apache/seatunnel-web/pull/208#issuecomment-2335128897 The primary modification in this Pull Request focuses on org.apache.seatunnel.app.utils.JobUtils#replaceJobConfigPlaceholders and its application. I am currently encounteri

Re: [PR] [Fix][Jdbc] Fix incorrect catalogName for getCatalogTable [seatunnel]

2024-09-07 Thread via GitHub
chl-wxp commented on code in PR #7595: URL: https://github.com/apache/seatunnel/pull/7595#discussion_r1748098409 ## seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-part-7/src/test/resources/jdbc_mysql_source_and_sink.conf: ## @@ -27,7 +27,8 @@ sour

Re: [PR] [Fix][Jdbc] Fix incorrect catalogName for getCatalogTable [seatunnel]

2024-09-07 Thread via GitHub
chl-wxp commented on code in PR #7595: URL: https://github.com/apache/seatunnel/pull/7595#discussion_r1748099029 ## seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-part-7/src/test/resources/jdbc_mysql_source_and_sink.conf: ## @@ -27,7 +27,8 @@ sour

Re: [I] [Improvement][Seatunnel-client] Support executing jobs from job configure string instead of file. [seatunnel]

2024-09-07 Thread via GitHub
arshadmohammad commented on issue #7600: URL: https://github.com/apache/seatunnel/issues/7600#issuecomment-2335425662 > How about we encrypt the configuration file content with Base64 Encryption is certainly important, but there's another issue at hand. Despite encrypting the data,

Re: [PR] [Fix][Jdbc] Fix incorrect catalogName for getCatalogTable [seatunnel]

2024-09-07 Thread via GitHub
Hisoka-X commented on code in PR #7595: URL: https://github.com/apache/seatunnel/pull/7595#discussion_r1748737851 ## seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-part-7/src/test/resources/jdbc_mysql_source_and_sink.conf: ## @@ -27,7 +27,8 @@ sou

Re: [PR] [Fix][Jdbc] Fix incorrect catalogName for getCatalogTable [seatunnel]

2024-09-07 Thread via GitHub
corgy-w commented on code in PR #7595: URL: https://github.com/apache/seatunnel/pull/7595#discussion_r1748835263 ## seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-common/src/test/java/org/apache/seatunnel/connectors/seatunnel/jdbc/AbstractJdbcIT.java: ###

Re: [PR] [Feature][zeta]Support exposing monitoring metrics by prometheus exporter protocol [seatunnel]

2024-09-07 Thread via GitHub
corgy-w commented on PR #7564: URL: https://github.com/apache/seatunnel/pull/7564#issuecomment-2336366795 PTAL @Hisoka-X @hailin0 -- 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 comm

Re: [PR] [Feature][Connector-V2][Hbase] implement hbase catalog [seatunnel]

2024-09-07 Thread via GitHub
zhangshenghang commented on code in PR #7516: URL: https://github.com/apache/seatunnel/pull/7516#discussion_r1749007997 ## seatunnel-connectors-v2/connector-hbase/src/main/java/org/apache/seatunnel/connectors/seatunnel/hbase/catalog/HbaseCatalog.java: ## @@ -0,0 +1,206 @@ +/* +

Re: [I] [Bug] [connector-paimon] NullPointerException when sink to paimon table [seatunnel]

2024-09-07 Thread via GitHub
github-actions[bot] commented on issue #7307: URL: https://github.com/apache/seatunnel/issues/7307#issuecomment-2336494776 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] Job Status update error when master switch [seatunnel]

2024-09-07 Thread via GitHub
github-actions[bot] closed issue #7286: [Bug] Job Status update error when master switch URL: https://github.com/apache/seatunnel/issues/7286 -- 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 spec

Re: [PR] [Fix][Jdbc] Fix incorrect catalogName for getCatalogTable [seatunnel]

2024-09-07 Thread via GitHub
chl-wxp commented on code in PR #7595: URL: https://github.com/apache/seatunnel/pull/7595#discussion_r1749032617 ## seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-part-7/src/test/resources/jdbc_mysql_source_and_sink.conf: ## @@ -27,7 +27,8 @@ sour

Re: [PR] [Feature][zeta]Support exposing monitoring metrics by prometheus exporter protocol [seatunnel]

2024-09-07 Thread via GitHub
Hisoka-X commented on code in PR #7564: URL: https://github.com/apache/seatunnel/pull/7564#discussion_r1749036199 ## config/seatunnel.yaml: ## @@ -34,3 +34,6 @@ seatunnel: namespace: /tmp/seatunnel/checkpoint_snapshot storage.type: hdfs fs.defaul

Re: [PR] [Improvement][Seatunnel-web] dom4j-1.6.1.jar has multiple CVEs, updat… [seatunnel-web]

2024-09-07 Thread via GitHub
BilwaST commented on PR #209: URL: https://github.com/apache/seatunnel-web/pull/209#issuecomment-2336543287 cc @arshadmohammad -- 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

Re: [PR] [Bug] [Seatunnel-web] [Metrics] NPE while fetching metrics from seatunnel-engine. [seatunnel-web]

2024-09-07 Thread via GitHub
arshadmohammad commented on PR #206: URL: https://github.com/apache/seatunnel-web/pull/206#issuecomment-2336565949 Thanks @Hisoka-X for reviewing and merging the PR -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U