general7078 commented on issue #2736:
URL:
https://github.com/apache/incubator-seatunnel/issues/2736#issuecomment-1437983478
> I use 2.3.0 and the developing 2.3.1, found the same bug. it can replace
with environment variable which use 'export param = param_val' from shell, but
it fails wh
general7078 commented on issue #2736:
URL:
https://github.com/apache/incubator-seatunnel/issues/2736#issuecomment-1437981672
I use 2.3.0 and the developing 2.3.1, found the same bug. it can replace
with environment variable which use 'export param = param_val' from shell, but
it fails when
TyrantLucifer commented on code in PR #4128:
URL:
https://github.com/apache/incubator-seatunnel/pull/4128#discussion_r1112626227
##
seatunnel-core/seatunnel-spark-starter/seatunnel-spark-3-starter/src/main/java/org/apache/seatunnel/core/starter/spark/execution/SinkExecuteProcessor.java
This is an automated email from the ASF dual-hosted git repository.
zongwen pushed a commit to branch cdc-multiple-table
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git
The following commit(s) were added to refs/heads/cdc-multiple-table by this
push:
new 5d2415117
ashulin merged PR #4176:
URL: https://github.com/apache/incubator-seatunnel/pull/4176
--
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..
ashulin opened a new pull request, #4176:
URL: https://github.com/apache/incubator-seatunnel/pull/4176
## 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 p
EricJoy2048 commented on issue #4150:
URL:
https://github.com/apache/incubator-seatunnel/issues/4150#issuecomment-1437891190
> SQL Parser and Generate PhysicalQueryPlan execution
It seems `SQL Parser and Generate PhysicalQueryPlan execution` is better way
for us.
--
This is an aut
EricJoy2048 commented on code in PR #4135:
URL:
https://github.com/apache/incubator-seatunnel/pull/4135#discussion_r1112534032
##
tools/update_modules_check/update_modules_check.py:
##
@@ -159,6 +159,37 @@ def get_sub_it_modules(modules, total_num, current_num):
output = o
This is an automated email from the ASF dual-hosted git repository.
zongwen pushed a commit to branch cdc-multiple-table
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git
The following commit(s) were added to refs/heads/cdc-multiple-table by this
push:
new c407ec5c2
ashulin merged PR #4175:
URL: https://github.com/apache/incubator-seatunnel/pull/4175
--
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..
TyrantLucifer commented on PR #4174:
URL:
https://github.com/apache/incubator-seatunnel/pull/4174#issuecomment-1437781173
> I have been tested,PTAL thanks. @TyrantLucifer
Give me some time. cc @john8628 Could you please help us test it?
--
This is an automated message from the Apac
hailin0 commented on code in PR #3679:
URL:
https://github.com/apache/incubator-seatunnel/pull/3679#discussion_r1112470799
##
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-starrocks-e2e/src/test/resources/starrocks-thrift-to-starrocks-streamload.conf:
##
@@ -21,12 +21,31 @
ashulin opened a new pull request, #4175:
URL: https://github.com/apache/incubator-seatunnel/pull/4175
## 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 p
wineternity commented on PR #4172:
URL:
https://github.com/apache/incubator-seatunnel/pull/4172#issuecomment-1437755956
> Can you add e2e test for this pr?
OK
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
liugddx commented on PR #4164:
URL:
https://github.com/apache/incubator-seatunnel/pull/4164#issuecomment-1437737669
you also should update release-note
https://github.com/apache/incubator-seatunnel/blob/dev/release-note.md
--
This is an automated message from the Apache Git Service.
To r
liugddx commented on PR #4174:
URL:
https://github.com/apache/incubator-seatunnel/pull/4174#issuecomment-1437730991
I have been tested,PTAL thanks. @TyrantLucifer
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
github-actions[bot] closed issue #2789: [Improve] [Connector-V2-Clickhouse]
Improve Clickhouse Connector
URL: https://github.com/apache/incubator-seatunnel/issues/2789
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
github-actions[bot] commented on issue #2789:
URL:
https://github.com/apache/incubator-seatunnel/issues/2789#issuecomment-1437698147
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.
--
github-actions[bot] commented on issue #3869:
URL:
https://github.com/apache/incubator-seatunnel/issues/3869#issuecomment-1437698103
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
rewerma commented on issue #4150:
URL:
https://github.com/apache/incubator-seatunnel/issues/4150#issuecomment-1437175304
>
H2 insert increase data to memory is not the main problem of worse
performance, but the JDBC executeQuery operation is.
As the benchmark, the operation of qu
TyrantLucifer merged PR #4155:
URL: https://github.com/apache/incubator-seatunnel/pull/4155
--
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-unsu
This is an automated email from the ASF dual-hosted git repository.
tyrantlucifer pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git
The following commit(s) were added to refs/heads/dev by this push:
new e109db903 [Improve][Shell] Remove
TyrantLucifer merged PR #4153:
URL: https://github.com/apache/incubator-seatunnel/pull/4153
--
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-unsu
TaoZex commented on code in PR #4153:
URL:
https://github.com/apache/incubator-seatunnel/pull/4153#discussion_r1112044239
##
seatunnel-examples/seatunnel-spark-connector-v2-example/src/main/resources/examples/spark.batch.conf:
##
@@ -1,98 +1,35 @@
-#
Review Comment:
Sorry,
liugddx commented on code in PR #4153:
URL:
https://github.com/apache/incubator-seatunnel/pull/4153#discussion_r1112035405
##
seatunnel-examples/seatunnel-spark-connector-v2-example/src/main/resources/examples/spark.batch.conf:
##
@@ -1,98 +1,35 @@
-#
Review Comment:
Why ch
TyrantLucifer commented on code in PR #4153:
URL:
https://github.com/apache/incubator-seatunnel/pull/4153#discussion_r1112034856
##
seatunnel-examples/seatunnel-spark-connector-v2-example/src/main/resources/examples/spark.batch.conf:
##
@@ -1,98 +1,35 @@
-#
-# Licensed to the A
Al-assad commented on PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#issuecomment-1437121401
>
>
I just submitted an update to the release-note and the approval status will
be dimissed again in the github mechanism, please re-add your approval, thanks
liugddx closed pull request #4159: [Hotfix][Zeta][Config] config
advanced-network-configuration
URL: https://github.com/apache/incubator-seatunnel/pull/4159
--
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
liugddx commented on PR #4159:
URL:
https://github.com/apache/incubator-seatunnel/pull/4159#issuecomment-1437117177
I will close this pr and refer to #4174
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
liugddx opened a new pull request, #4174:
URL: https://github.com/apache/incubator-seatunnel/pull/4174
## 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 p
TyrantLucifer commented on code in PR #4153:
URL:
https://github.com/apache/incubator-seatunnel/pull/4153#discussion_r1112025381
##
bin/install-plugin.sh:
##
@@ -43,61 +43,18 @@ if [ ! -d ${SEATUNNEL_HOME}/connectors ];
echo "create connectors directory"
fi
-# create
liugddx commented on PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#issuecomment-1437101584
> > Update
https://github.com/apache/incubator-seatunnel/blob/dev/seatunnel-dist/pom.xml
>
> Resolved 😃.
You should update `release-note.md` for this pr.
--
T
Al-assad commented on PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#issuecomment-1437087172
> Update
https://github.com/apache/incubator-seatunnel/blob/dev/seatunnel-dist/pom.xml
Resolved 😃.
--
This is an automated message from the Apache Git Service.
To r
ic4y closed pull request #4088: [Feature][Zeta] Enable Task thread share
URL: https://github.com/apache/incubator-seatunnel/pull/4088
--
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
ic4y opened a new pull request, #4173:
URL: https://github.com/apache/incubator-seatunnel/pull/4173
## 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 pack
liugddx commented on code in PR #4153:
URL:
https://github.com/apache/incubator-seatunnel/pull/4153#discussion_r1112004533
##
bin/install-plugin.sh:
##
@@ -43,61 +43,18 @@ if [ ! -d ${SEATUNNEL_HOME}/connectors ];
echo "create connectors directory"
fi
-# create the fl
wineternity opened a new pull request, #4172:
URL: https://github.com/apache/incubator-seatunnel/pull/4172
## Purpose of this pull request
## Check list
* [x] Code changed are covered with tests, or it does not need tests for
reason:
* [x] If any new Jar bina
laglangyue commented on issue #4150:
URL:
https://github.com/apache/incubator-seatunnel/issues/4150#issuecomment-1437066588
For H2, we need to write data into H2, and then query the results. With the
increase of data volume, the performance will be much worse. For calcite, IO is
not need.
TyrantLucifer commented on PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#issuecomment-1437051503
> > Update
https://github.com/apache/incubator-seatunnel/blob/dev/seatunnel-dist/pom.xml
>
> Sorry my fault, you also should update release-note
https://github.c
TyrantLucifer commented on PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#issuecomment-1437051066
> Update
https://github.com/apache/incubator-seatunnel/blob/dev/seatunnel-dist/pom.xml
Sorry my fault, you also should update release-note
https://github.com/apa
This is an automated email from the ASF dual-hosted git repository.
songjian pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel-web.git
The following commit(s) were added to refs/heads/main by this push:
new c1c0f862 [Feat][UI] Adjust menu s
songjianet merged PR #25:
URL: https://github.com/apache/incubator-seatunnel-web/pull/25
--
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-unsubsc
Al-assad commented on PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#issuecomment-1437029872
@liugddx @TyrantLucifer Thanks for your suggestions, please take a look
agian.
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
songjianet opened a new pull request, #25:
URL: https://github.com/apache/incubator-seatunnel-web/pull/25
## 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 binar
TaoZex commented on code in PR #4153:
URL:
https://github.com/apache/incubator-seatunnel/pull/4153#discussion_r947792
##
bin/install-plugin.sh:
##
@@ -43,57 +43,16 @@ if [ ! -d ${SEATUNNEL_HOME}/connectors ];
echo "create connectors directory"
fi
-# create the fli
liugddx commented on PR #4159:
URL:
https://github.com/apache/incubator-seatunnel/pull/4159#issuecomment-1436983491
> This pull request maybe cause this error:
>
>
https://github.com/apache/incubator-seatunnel/actions/runs/4223196739/jobs/7332744419#step:5:44330
>
>
https://g
TyrantLucifer closed issue #4163: [Feature][format] Add avro format for
Seatunnel
URL: https://github.com/apache/incubator-seatunnel/issues/4163
--
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 s
TyrantLucifer commented on issue #4163:
URL:
https://github.com/apache/incubator-seatunnel/issues/4163#issuecomment-1436971326
This requirement has been discussed in
https://github.com/apache/incubator-seatunnel/issues/3120, please send an
e-mail to d...@seatunnel.apache.org and discuss wi
TyrantLucifer commented on PR #4160:
URL:
https://github.com/apache/incubator-seatunnel/pull/4160#issuecomment-1436958147
Could you please submit this pull request to dev branch? Becase the code
could not be changed when release process has done.
This bug will be fixed in 2.3.1-release.
TyrantLucifer closed pull request #4160: [Bug][Connector-V2] Field aliases are
not supported in the query of jdbc source.
URL: https://github.com/apache/incubator-seatunnel/pull/4160
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
This is an automated email from the ASF dual-hosted git repository.
songjian pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel-web.git
The following commit(s) were added to refs/heads/main by this push:
new a51089ed [Feat][UI] Complete styl
songjianet merged PR #24:
URL: https://github.com/apache/incubator-seatunnel-web/pull/24
--
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-unsubsc
songjianet opened a new pull request, #24:
URL: https://github.com/apache/incubator-seatunnel-web/pull/24
## Purpose of this pull request
![1676896683658](ht
TyrantLucifer commented on PR #4159:
URL:
https://github.com/apache/incubator-seatunnel/pull/4159#issuecomment-1436945207
This pull request maybe cause this error:
https://github.com/apache/incubator-seatunnel/actions/runs/4223196739/jobs/7332744419#step:5:44330
--
This is an auto
rewerma commented on issue #4150:
URL:
https://github.com/apache/incubator-seatunnel/issues/4150#issuecomment-1436924220
> >
>
> Is there any benchmark data?
Read, Transform and Write 2,000,000 rows benchmark data, Implements with:
H2:
![30dd12d860fc9ce2db99a27e45e17746
TyrantLucifer commented on code in PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#discussion_r870814
##
seatunnel-connectors-v2/connector-http/connector-http-github/src/main/java/org/apache/seatunnel/connectors/seatunnel/github/GithubPlugin.java:
##
liugddx commented on PR #4022:
URL:
https://github.com/apache/incubator-seatunnel/pull/4022#issuecomment-1436859698
Can this PR continue?
--
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
rewerma commented on issue #4150:
URL:
https://github.com/apache/incubator-seatunnel/issues/4150#issuecomment-1436857532
> > > > How about add a Simple SQL transform plugin with internal mem SQL
engine like H2DB, SQLite or Calcite. #4149
> > >
> > >
> > > H2 is not a good impleme
liugddx commented on issue #4150:
URL:
https://github.com/apache/incubator-seatunnel/issues/4150#issuecomment-1436855725
>
Is there any benchmark data?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
rewerma commented on issue #4150:
URL:
https://github.com/apache/incubator-seatunnel/issues/4150#issuecomment-1436851149
I implement the SQL engine by H2, Calcite and Internal implements.
The Simple SQL physical query plan is simple, only three steps:
PhysicalScanTable, PhysicalFilter an
liugddx commented on code in PR #4155:
URL:
https://github.com/apache/incubator-seatunnel/pull/4155#discussion_r845409
##
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-http-e2e/src/test/java/org/apache/seatunnel/e2e/connector/http/HttpGithubIT.java:
##
@@ -0,0 +1,31 @@
sunxiaojian commented on issue #4163:
URL:
https://github.com/apache/incubator-seatunnel/issues/4163#issuecomment-1436835379
You can assign it to me
--
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
liugddx commented on code in PR #4153:
URL:
https://github.com/apache/incubator-seatunnel/pull/4153#discussion_r831155
##
bin/install-plugin.sh:
##
@@ -43,57 +43,16 @@ if [ ! -d ${SEATUNNEL_HOME}/connectors ];
echo "create connectors directory"
fi
-# create the fl
dijiekstra commented on code in PR #23:
URL:
https://github.com/apache/incubator-seatunnel-web/pull/23#discussion_r817546
##
seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/service/impl/UserServiceImpl.java:
##
@@ -142,6 +145,10 @@ public UserSimpleInf
dijiekstra merged PR #21:
URL: https://github.com/apache/incubator-seatunnel-web/pull/21
--
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-unsubsc
This is an automated email from the ASF dual-hosted git repository.
dijiekstra pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel-web.git
The following commit(s) were added to refs/heads/main by this push:
new 96ace88c [Fix][API] Fix that th
This is an automated email from the ASF dual-hosted git repository.
zongwen pushed a commit to branch cdc-multiple-table
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git
The following commit(s) were added to refs/heads/cdc-multiple-table by this
push:
new d144e176b
ashulin merged PR #4170:
URL: https://github.com/apache/incubator-seatunnel/pull/4170
--
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..
This is an automated email from the ASF dual-hosted git repository.
zongwen pushed a commit to branch cdc-multiple-table
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git
The following commit(s) were added to refs/heads/cdc-multiple-table by this
push:
new 1636afad0
ashulin merged PR #4171:
URL: https://github.com/apache/incubator-seatunnel/pull/4171
--
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..
ashulin opened a new pull request, #4171:
URL: https://github.com/apache/incubator-seatunnel/pull/4171
## 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 p
ashulin opened a new pull request, #4170:
URL: https://github.com/apache/incubator-seatunnel/pull/4170
## 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 p
ashulin merged PR #4168:
URL: https://github.com/apache/incubator-seatunnel/pull/4168
--
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..
This is an automated email from the ASF dual-hosted git repository.
zongwen pushed a commit to branch cdc-multiple-table
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git
The following commit(s) were added to refs/heads/cdc-multiple-table by this
push:
new 95e3cbf87
This is an automated email from the ASF dual-hosted git repository.
zongwen pushed a commit to branch cdc-multiple-table
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git
The following commit(s) were added to refs/heads/cdc-multiple-table by this
push:
new c892dea5c
ashulin merged PR #4169:
URL: https://github.com/apache/incubator-seatunnel/pull/4169
--
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..
ashulin merged PR #4157:
URL: https://github.com/apache/incubator-seatunnel/pull/4157
--
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..
liugddx commented on PR #4159:
URL:
https://github.com/apache/incubator-seatunnel/pull/4159#issuecomment-1436578148
> This pull request maybe cause #4154
I have communicated with him and the problem has been resolved. CC :
@john8628
--
This is an automated message from the Apache
TyrantLucifer commented on PR #4159:
URL:
https://github.com/apache/incubator-seatunnel/pull/4159#issuecomment-1436574414
link #4154
--
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
liugddx closed issue #4154: [bug][zeta] seatunnel -cluster
URL: https://github.com/apache/incubator-seatunnel/issues/4154
--
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 un
liugddx commented on issue #4154:
URL:
https://github.com/apache/incubator-seatunnel/issues/4154#issuecomment-1436544778
Ok, I will close this issue first, if there are still problems, you can
reopen it. cc: @TyrantLucifer @EricJoy2048 @CalvinKirs @Hisoka-X
--
This is an automated messa
john8628 commented on issue #4154:
URL:
https://github.com/apache/incubator-seatunnel/issues/4154#issuecomment-1436532724
change the member-server-socket-endpoint-config: port can solve this problem
--
This is an automated message from the Apache Git Service.
To respond to the message, p
ashulin opened a new pull request, #4169:
URL: https://github.com/apache/incubator-seatunnel/pull/4169
## 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 p
ashulin opened a new pull request, #4168:
URL: https://github.com/apache/incubator-seatunnel/pull/4168
## 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 p
ashulin commented on PR #4157:
URL:
https://github.com/apache/incubator-seatunnel/pull/4157#issuecomment-1436519788
@hailin0 @TyrantLucifer 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
85 matches
Mail list logo
