ruanwenjun opened a new issue, #5571:
URL: https://github.com/apache/seatunnel/issues/5571
### 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
XiaoJiang521 closed pull request #5533: [imporve] dm use schema and table
URL: https://github.com/apache/seatunnel/pull/5533
--
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
XiaoJiang521 opened a new pull request, #5572:
URL: https://github.com/apache/seatunnel/pull/5572
### Purpose of this pull request
1. mysql pg sqlserver oracle and dm auto create table add constraintKey
2. dm catalog and dm auto create table
### Does this
whhe commented on code in PR #5517:
URL: https://github.com/apache/seatunnel/pull/5517#discussion_r1338287551
##
seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/catalog/oracle/OracleCatalog.java:
##
@@ -129,12 +129,12 @@ protec
Hisoka-X commented on code in PR #4918:
URL: https://github.com/apache/seatunnel/pull/4918#discussion_r1338375371
##
seatunnel-core/seatunnel-flink-starter/seatunnel-flink-starter-common/src/main/java/org/apache/seatunnel/core/starter/flink/execution/FlinkAbstractPluginExecuteProcessor.
Hisoka-X commented on code in PR #4918:
URL: https://github.com/apache/seatunnel/pull/4918#discussion_r1338375371
##
seatunnel-core/seatunnel-flink-starter/seatunnel-flink-starter-common/src/main/java/org/apache/seatunnel/core/starter/flink/execution/FlinkAbstractPluginExecuteProcessor.
xiamidavid00 opened a new pull request, #5573:
URL: https://github.com/apache/seatunnel/pull/5573
Add documents for Zeta engine running on Docker
--
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
XiaoJiang521 commented on code in PR #5517:
URL: https://github.com/apache/seatunnel/pull/5517#discussion_r1338561043
##
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-part-1/src/test/java/org/apache/seatunnel/connectors/seatunnel/jdbc/JdbcOracleLowerTabl
XiaoJiang521 commented on code in PR #5517:
URL: https://github.com/apache/seatunnel/pull/5517#discussion_r1338564500
##
seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/catalog/oracle/OracleCatalog.java:
##
@@ -129,12 +129,12 @
Hisoka-X merged PR #5568:
URL: https://github.com/apache/seatunnel/pull/5568
--
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
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 fd7727869f [Improve][Test] Remove docker image afte
hailin0 commented on PR #5550:
URL: https://github.com/apache/seatunnel/pull/5550#issuecomment-1737383280
Please check ci error
--
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
Hisoka-X commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338601437
##
docs/en/concept/schema-feature.md:
##
@@ -2,9 +2,55 @@
## Why we need schema
-Some NoSQL databases or message queue are not strongly limited schema, so the
Hisoka-X opened a new pull request, #5574:
URL: https://github.com/apache/seatunnel/pull/5574
### Purpose of this pull request
The build status in `README.md` not update when we move CI to fork
repository.
This PR change it to new build yml file, to sync build status in README.
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338758293
##
docs/en/concept/schema-feature.md:
##
@@ -26,9 +72,111 @@ Some NoSQL databases or message queue are not strongly
limited schema, so the sc
| map | `java
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338760687
##
docs/en/concept/schema-feature.md:
##
@@ -26,9 +72,111 @@ Some NoSQL databases or message queue are not strongly
limited schema, so the sc
| map | `java
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338766058
##
seatunnel-api/src/main/java/org/apache/seatunnel/api/table/catalog/CatalogTableUtil.java:
##
@@ -117,7 +113,7 @@ public static List getCatalogTables(Config
con
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338765332
##
docs/en/concept/schema-feature.md:
##
@@ -2,9 +2,55 @@
## Why we need schema
-Some NoSQL databases or message queue are not strongly limited schema, so the
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338768819
##
seatunnel-api/src/main/java/org/apache/seatunnel/api/table/catalog/CatalogTableUtil.java:
##
@@ -184,16 +180,31 @@ public static List
getCatalogTablesFromConfi
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338769156
##
seatunnel-api/src/main/java/org/apache/seatunnel/api/table/catalog/schema/ReadonlyConfigParser.java:
##
@@ -0,0 +1,378 @@
+/*
Review Comment:
I have removed
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338770738
##
seatunnel-api/src/test/java/org/apache/seatunnel/api/table/catalog/CatalogTableUtilTest.java:
##
@@ -51,7 +51,7 @@ public void testSimpleSchemaParse() throws
F
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338772056
##
seatunnel-connectors-v2/connector-fake/src/test/java/org/apache/seatunnel/connectors/seatunnel/fake/source/FakeDataGeneratorTest.java:
##
@@ -111,7 +111,9 @@ pu
ruanwenjun commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1338965787
##
seatunnel-common/src/main/java/org/apache/seatunnel/common/utils/JsonUtils.java:
##
@@ -178,6 +180,24 @@ public static Map toMap(JsonNode jsonNode)
{
github-actions[bot] commented on issue #5337:
URL: https://github.com/apache/seatunnel/issues/5337#issuecomment-1738273564
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-actions[bot] closed issue #5340: [Bug] [MongoDB-CDC] 数据中有字段值为Null 直接异常.
URL: https://github.com/apache/seatunnel/issues/5340
--
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
JohnZp opened a new issue, #5575:
URL: https://github.com/apache/seatunnel/issues/5575
### 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
whhe commented on PR #5439:
URL: https://github.com/apache/seatunnel/pull/5439#issuecomment-1738357403
I tried to enable the oceanbase it case with the new docker image, and there
are still some issues about the container startup, which I will fix in another
pr. The label is removed by the
Hisoka-X commented on code in PR #5561:
URL: https://github.com/apache/seatunnel/pull/5561#discussion_r1339451621
##
docs/en/connector-v2/source/Http.md:
##
@@ -42,24 +42,31 @@ They can be downloaded via install-plugin.sh or from the
Maven central repositor
## Source Options
Hisoka-X commented on PR #5561:
URL: https://github.com/apache/seatunnel/pull/5561#issuecomment-1738364169
@xiaofan2022 Also please add e2e test case for this new feature, you can
refer
https://github.com/apache/seatunnel/tree/dev/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-http-e2e/
xiamidavid00 closed pull request #5573: [Feature][Doc] Add Docker Doc #5513
URL: https://github.com/apache/seatunnel/pull/5573
--
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.
T
Hisoka-X commented on PR #5561:
URL: https://github.com/apache/seatunnel/pull/5561#issuecomment-1738364731
cc @ruanwenjun @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 co
xiaofan2022 commented on code in PR #5561:
URL: https://github.com/apache/seatunnel/pull/5561#discussion_r1339464449
##
docs/en/connector-v2/source/Http.md:
##
@@ -42,24 +42,31 @@ They can be downloaded via install-plugin.sh or from the
Maven central repositor
## Source Opti
xiamidavid00 opened a new pull request, #5576:
URL: https://github.com/apache/seatunnel/pull/5576
Add documents for Zeta engine running on Docker
--
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 t
xiamidavid00 closed pull request #5576: [Feature][Doc] Add Docker Doc #5513
URL: https://github.com/apache/seatunnel/pull/5576
--
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.
T
xiamidavid00 opened a new pull request, #5577:
URL: https://github.com/apache/seatunnel/pull/5577
Add Docker Doc
--
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 unsubscr
xiamidavid00 closed pull request #5577: [Feature][Doc] Add Docker Doc #5513
URL: https://github.com/apache/seatunnel/pull/5577
--
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.
T
liutaobigdata opened a new issue, #5578:
URL: https://github.com/apache/seatunnel/issues/5578
### 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 happ
ic4y merged PR #5551:
URL: https://github.com/apache/seatunnel/pull/5551
--
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
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 512d982ee3 Throw IllegalArgumentException when find m
JohnZp opened a new pull request, #5579:
URL: https://github.com/apache/seatunnel/pull/5579
### Purpose of this pull request
Fix NPE, it introduced by https://github.com/apache/seatunnel/issues/5575
### Does this PR introduce _any_ user-facing change?
No
##
Hisoka-X commented on code in PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#discussion_r1339578076
##
seatunnel-api/src/main/java/org/apache/seatunnel/api/table/catalog/CatalogTableUtil.java:
##
@@ -46,16 +38,20 @@
import java.util.ArrayList;
import java.util.Co
Hisoka-X commented on PR #5564:
URL: https://github.com/apache/seatunnel/pull/5564#issuecomment-1738535054
Any demo we can directly run? I find we add this new feature but we don't
have any e2e demo to prove it worked.
--
This is an automated message from the Apache Git Service.
To respon
xiamidavid00 closed pull request #5577: [Feature][Doc] Add Docker Doc #5513
URL: https://github.com/apache/seatunnel/pull/5577
--
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.
T
Hisoka-X commented on PR #5572:
URL: https://github.com/apache/seatunnel/pull/5572#issuecomment-1738546704
Shall we add some test case?
--
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 specifi
XiaoJiang521 commented on PR #5517:
URL: https://github.com/apache/seatunnel/pull/5517#issuecomment-1738551693
> Could you add some comment like this?

>
> Shall we add some
Hisoka-X commented on PR #5579:
URL: https://github.com/apache/seatunnel/pull/5579#issuecomment-1738551781
Could you open your fork repository action follow the guide?

--
This is an a
46 matches
Mail list logo