[PR] Update .asf.yaml [incubator-seata-k8s]

2024-01-04 Thread via GitHub
funky-eyes opened a new pull request, #7: URL: https://github.com/apache/incubator-seata-k8s/pull/7 (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 unsubscr

[I] similar table names cause "Not support the table has the same column name with different case yet" [incubator-seata]

2024-01-04 Thread via GitHub
SpectralKylin opened a new issue, #6241: URL: https://github.com/apache/incubator-seata/issues/6241 ### Ⅰ. Issue Description similar table names cause "Not support the table has the same column name with different case yet". such as:bill_test、bill_test_data_0. ### Ⅱ. Describe wh

[I] gorm使用seata分布式事务报空指针 [incubator-seata-go]

2024-01-04 Thread via GitHub
vlon opened a new issue, #643: URL: https://github.com/apache/incubator-seata-go/issues/643 ![a1048de852550680e15c5442fae59e3c](https://github.com/apache/incubator-seata-go/assets/29192868/b94b930a-297e-435b-bf88-cd8a5f99f176) **What happened**: **What you expected to h

Re: [I] java和go服务共同组成分布式事务测试 [incubator-seata-go]

2024-01-04 Thread via GitHub
vlon commented on issue #501: URL: https://github.com/apache/incubator-seata-go/issues/501#issuecomment-1878261777 这个测试有结果了吗,我会在项目中用到java和go,java会作为业务层实现语言,go作为低层数据层的实现,java会grpc调用go,需要用到seata来做分布式事务 -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] optimize: add unified footers [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly merged PR #752: URL: https://github.com/apache/incubator-seata-website/pull/752 -- 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: dev-unsubscr...

Re: [PR] optimize: branch register resource only at RM server end [incubator-seata]

2024-01-04 Thread via GitHub
wt-better commented on PR #5399: URL: https://github.com/apache/incubator-seata/pull/5399#issuecomment-1878223179 LGTM -- 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

[PR] optimize: add unified footers [incubator-seata-website]

2024-01-04 Thread via GitHub
liuqiufeng opened a new pull request, #752: URL: https://github.com/apache/incubator-seata-website/pull/752 ## What is the purpose of the change X ## Brief changelog X Follow this checklist to help us incorporate your contribution quickly and easily:

Re: [I] Database read-write separation, RM registration reads are not from the main database [incubator-seata]

2024-01-04 Thread via GitHub
Python-IBM closed issue #6236: Database read-write separation, RM registration reads are not from the main database URL: https://github.com/apache/incubator-seata/issues/6236 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [I] The console needs to display the version number [incubator-seata]

2024-01-04 Thread via GitHub
liuqiufeng commented on issue #6240: URL: https://github.com/apache/incubator-seata/issues/6240#issuecomment-1878189831 plz assign 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 to the speci

[I] The console needs to display the version number [incubator-seata]

2024-01-04 Thread via GitHub
slievrly opened a new issue, #6240: URL: https://github.com/apache/incubator-seata/issues/6240 ## Why you need it? Is your feature request related to a problem? Please describe in details console needs to display the version number ## How it could be? ![i

Re: [I] Database read-write separation, RM registration reads are not from the main database [incubator-seata]

2024-01-04 Thread via GitHub
Python-IBM commented on issue #6236: URL: https://github.com/apache/incubator-seata/issues/6236#issuecomment-1878105682 I have a question, why does seata rm register resources to read the jdbc URL of slave mysql? -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] docs: add template files for community [incubator-seata-ctl]

2024-01-04 Thread via GitHub
slievrly commented on code in PR #3: URL: https://github.com/apache/incubator-seata-ctl/pull/3#discussion_r1442472509 ## .github/ISSUE_TEMPLATE /BUG_REPORT.md: ## @@ -0,0 +1,45 @@ +--- +name: Bug Report + +about: If you would like to report an issue to Seata, please use this

Re: [PR] docs: add template files for community [incubator-seata-ctl]

2024-01-04 Thread via GitHub
slievrly commented on code in PR #3: URL: https://github.com/apache/incubator-seata-ctl/pull/3#discussion_r1442472248 ## .github/ISSUE_TEMPLATE /BUG_REPORT.md: ## @@ -0,0 +1,45 @@ +--- +name: Bug Report + +about: If you would like to report an issue to Seata, please use this

Re: [PR] optimize: update security policy, disclaimer and notice [incubator-seata]

2024-01-04 Thread via GitHub
codecov-commenter commented on PR #6239: URL: https://github.com/apache/incubator-seata/pull/6239#issuecomment-1878082087 ## [Codecov](https://app.codecov.io/gh/apache/incubator-seata/pull/6239?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&ut

[PR] optimize: update security policy, disclaimer and notice [incubator-seata]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #6239: URL: https://github.com/apache/incubator-seata/pull/6239 - [ ] I have registered the PR [changes](../changes). ### Ⅰ. Describe what this PR did update security policy, disclaimer and notice ### Ⅱ. Does this pull request f

Re: [PR] optimize: optimize some files [incubator-seata]

2024-01-04 Thread via GitHub
codecov-commenter commented on PR #6238: URL: https://github.com/apache/incubator-seata/pull/6238#issuecomment-1878069744 ## [Codecov](https://app.codecov.io/gh/apache/incubator-seata/pull/6238?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&ut

[PR] optimize: optimize some files [incubator-seata]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #6238: URL: https://github.com/apache/incubator-seata/pull/6238 - [ ] I have registered the PR [changes](../changes). ### Ⅰ. Describe what this PR did optimize some files ### Ⅱ. Does this pull request fix one issue?

Re: [I] Seata2.0.0服务端无报错,客户端执行update空指针异常 [incubator-seata]

2024-01-04 Thread via GitHub
menggangaaa commented on issue #6237: URL: https://github.com/apache/incubator-seata/issues/6237#issuecomment-1878056386 已解决 原因:Maven客户端依赖顶级pom的1.6.1版本被Spring Cloud的版本依赖2021.0.1管理到1.4.2了 解决方案,升级Spring Cloud版本到2021.0.5 -- This is an automated message from the Apache Git Service. To r

Re: [I] Seata2.0.0服务端无报错,客户端执行update空指针异常 [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on issue #6237: URL: https://github.com/apache/incubator-seata/issues/6237#issuecomment-1878050105 The version is not 2.0.0 -- 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

Re: [PR] optimize: add disclaimer and notice [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly merged PR #750: URL: https://github.com/apache/incubator-seata-website/pull/750 -- 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: dev-unsubscr...

[PR] optimize: replace urls [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #751: URL: https://github.com/apache/incubator-seata-website/pull/751 ## What is the purpose of the change replace urls ## Brief changelog replace urls Follow this checklist to help us incorporate your contribution quickly and e

Re: [PR] doc: add《Bridging Data and Applications》blog [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly merged PR #749: URL: https://github.com/apache/incubator-seata-website/pull/749 -- 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: dev-unsubscr...

Re: [I] Database read-write separation, RM registration reads are not from the main database [incubator-seata]

2024-01-04 Thread via GitHub
Python-IBM closed issue #6236: Database read-write separation, RM registration reads are not from the main database URL: https://github.com/apache/incubator-seata/issues/6236 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[PR] optimize: add disclaimer and notice [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #750: URL: https://github.com/apache/incubator-seata-website/pull/750 ## What is the purpose of the change add disclaimer and notice ## Brief changelog add disclaimer and notice Follow this checklist to help us incorporate your c

Re: [PR] optimize : load SeataSerializer by version [incubator-seata]

2024-01-04 Thread via GitHub
Bughue commented on code in PR #6208: URL: https://github.com/apache/incubator-seata/pull/6208#discussion_r1442439604 ## serializer/seata-serializer-seata/src/main/java/io/seata/serializer/seata/SeataSerializer.java: ## @@ -16,52 +16,133 @@ */ package io.seata.serializer.seat

Re: [PR] optimize : load SeataSerializer by version [incubator-seata]

2024-01-04 Thread via GitHub
Bughue commented on code in PR #6208: URL: https://github.com/apache/incubator-seata/pull/6208#discussion_r1442439517 ## serializer/seata-serializer-seata/src/main/java/io/seata/serializer/seata/SeataSerializer.java: ## @@ -16,52 +16,133 @@ */ package io.seata.serializer.seat

[I] Seata2.0.0服务端无报错,客户端执行update空指针异常 [incubator-seata]

2024-01-04 Thread via GitHub
menggangaaa opened a new issue, #6237: URL: https://github.com/apache/incubator-seata/issues/6237 Seata服务端版本2.0.0 Maven客户端依赖1.6.1 JDK8 Spring JDBC Template 在客户端 执行了 update表的语句,报错空指针,重复执行均报错 以下是堆栈 `09:52:22 [http-nio-8012-exec-8] INFO i.s.tm.api.DefaultGlobalTransaction - S

Re: [I] MaxGlobalSessionSize not MaxBranchSessionSize [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on issue #6189: URL: https://github.com/apache/incubator-seata/issues/6189#issuecomment-1877998537 > 我觉得size这个变量名称没必要改,因为 > > 1. 具有通用性,后续如果变了单位,难道要再次兼容吗? > 2. 项目中很多地方都用这个size,要改的话需要全部改,风险比较大 >建议在注释中写清楚这个单位是什么,然后改一下log打印的内容就可以了 你说的也是一种方式,先把错误的提示改了,

Re: [PR] optimize : load SeataSerializer by version [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on code in PR #6208: URL: https://github.com/apache/incubator-seata/pull/6208#discussion_r1442424823 ## serializer/seata-serializer-seata/src/main/java/io/seata/serializer/seata/SeataSerializer.java: ## @@ -16,52 +16,133 @@ */ package io.seata.serializer.

Re: [PR] optimize : load SeataSerializer by version [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on code in PR #6208: URL: https://github.com/apache/incubator-seata/pull/6208#discussion_r1442423580 ## serializer/seata-serializer-seata/src/main/java/io/seata/serializer/seata/SeataSerializer.java: ## @@ -16,52 +16,133 @@ */ package io.seata.serializer.

Re: [I] Database read-write separation, RM registration reads are not from the main database [incubator-seata]

2024-01-04 Thread via GitHub
Python-IBM commented on issue #6236: URL: https://github.com/apache/incubator-seata/issues/6236#issuecomment-1877992795 > 使用sharding-jdbc请不要开启seata的数据源代理,具体使用方式参见他们的文档 Please do not open seata's data source proxy when using sharding-jdbc. For details, please refer to their documentation

Re: [I] Database read-write separation, RM registration reads are not from the main database [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on issue #6236: URL: https://github.com/apache/incubator-seata/issues/6236#issuecomment-1877990385 使用sharding-jdbc请不要开启seata的数据源代理,具体使用方式参见他们的文档 Please do not open seata's data source proxy when using sharding-jdbc. For details, please refer to their documentation

Re: [PR] optimize: add ASF incubator footer [incubator-seata-website]

2024-01-04 Thread via GitHub
xingfudeshi merged PR #748: URL: https://github.com/apache/incubator-seata-website/pull/748 -- 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: dev-unsubscr

[I] Database read-write separation, RM registration reads are not from the main database [incubator-seata]

2024-01-04 Thread via GitHub
Python-IBM opened a new issue, #6236: URL: https://github.com/apache/incubator-seata/issues/6236 - [ ] I have searched the [issues](https://github.com/seata/seata/issues) of this repository and believe that this is not a duplicate. ### Ⅰ. Issue Description Implement MySQL r

[PR] doc: add《Bridging Data and Applications》doc [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #749: URL: https://github.com/apache/incubator-seata-website/pull/749 ## What is the purpose of the change doc: add《Bridging Data and Applications》 ## Brief changelog doc: add《Bridging Data and Applications》 Follow this checklist

Re: [I] MaxGlobalSessionSize not MaxBranchSessionSize [incubator-seata]

2024-01-04 Thread via GitHub
Jamesmahaitao commented on issue #6189: URL: https://github.com/apache/incubator-seata/issues/6189#issuecomment-1877292211 我觉得size这个变量名称没必要改,因为 1. 具有通用性,后续如果变了单位,难道要再次兼容吗? 2. 项目中很多地方都用这个size,要改的话需要全部改,风险比较大 建议在注释中写清楚这个单位是什么,然后改一下log打印的内容就可以了 -- This is an automated message from

Re: [I] Compensatory and forward operations in seata-go Saga [incubator-seata-go]

2024-01-04 Thread via GitHub
xjlgod commented on issue #642: URL: https://github.com/apache/incubator-seata-go/issues/642#issuecomment-1877236001 please assign 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 to the speci

Re: [I] seata saga 主体框架搭建 [incubator-seata-go]

2024-01-04 Thread via GitHub
wt-better closed issue #618: seata saga 主体框架搭建 URL: https://github.com/apache/incubator-seata-go/issues/618 -- 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-ma

Re: [I] seata saga 主体框架搭建 [incubator-seata-go]

2024-01-04 Thread via GitHub
wt-better closed issue #618: seata saga 主体框架搭建 URL: https://github.com/apache/incubator-seata-go/issues/618 -- 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-ma

Re: [I] seata saga 主体框架搭建 [incubator-seata-go]

2024-01-04 Thread via GitHub
wt-better commented on issue #618: URL: https://github.com/apache/incubator-seata-go/issues/618#issuecomment-1877211785 see feature/saga branch -- 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

Re: [D] 官网合规改造 [incubator-seata]

2024-01-04 Thread via GitHub
GitHub user liuqiufeng edited a discussion: 官网合规改造 合规要求:https://incubator.apache.org/guides/sites.html 潜在问题:https://incubator.apache.org/clutch/seata.html ## 免责声明 网站和所有文档中包含明确的免责声明,表明它们正在孵化中。使用以下文本作为所有免责声明(适当替换加粗的短语): Apache **Podling-Name** is an effort undergoing incubation at The Apache So

[PR] optimize: add ASF incubator footer [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #748: URL: https://github.com/apache/incubator-seata-website/pull/748 ## What is the purpose of the change add ASF incubator footer ## Brief changelog add ASF incubator footer Follow this checklist to help us incorporate your con

Re: [PR] optimize: add ASF doc reference [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly merged PR #747: URL: https://github.com/apache/incubator-seata-website/pull/747 -- 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: dev-unsubscr...

[PR] optimize: add ASF doc reference [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #747: URL: https://github.com/apache/incubator-seata-website/pull/747 ## What is the purpose of the change add ASF doc reference ## Brief changelog add ASF doc reference Follow this checklist to help us incorporate your contribut

Re: [I] seata deploy workflow failed. [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly closed issue #745: seata deploy workflow failed. URL: https://github.com/apache/incubator-seata-website/issues/745 -- 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 u

Re: [PR] optimize: fix deploy workflow failed [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly merged PR #746: URL: https://github.com/apache/incubator-seata-website/pull/746 -- 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: dev-unsubscr...

Re: [PR] Test: increase seata-core module unit test coverage [incubator-seata]

2024-01-04 Thread via GitHub
lightClouds917 closed pull request #6168: Test: increase seata-core module unit test coverage URL: https://github.com/apache/incubator-seata/pull/6168 -- 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

Re: [PR] optimize: RM TM startup connect server fail fast [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes merged PR #6004: URL: https://github.com/apache/incubator-seata/pull/6004 -- 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: dev-unsubscr...@sea

Re: [I] support active check of service availability at client startup [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes closed issue #5484: support active check of service availability at client startup URL: https://github.com/apache/incubator-seata/issues/5484 -- 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

Re: [PR] optimize: RM TM startup connect server fail fast [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on code in PR #6004: URL: https://github.com/apache/incubator-seata/pull/6004#discussion_r1441548112 ## core/src/test/java/io/seata/core/rpc/netty/TmNettyClientTest.java: ## @@ -123,6 +135,24 @@ public void setApplicationId() throws Exception { } +

Re: [PR] optimize: RM TM startup connect server fail fast [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on code in PR #6004: URL: https://github.com/apache/incubator-seata/pull/6004#discussion_r1441545616 ## core/src/test/java/io/seata/core/rpc/netty/RmNettyClientTest.java: ## @@ -47,6 +78,24 @@ public void assertGetInstanceAfterDestroy() { assertTrue

[PR] optimize: fix deploy workflow failed [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly opened a new pull request, #746: URL: https://github.com/apache/incubator-seata-website/pull/746 ## What is the purpose of the change fix deploy workflow failed ## Brief changelog fix deploy workflow failed fix #745 Follow this checklist to help us

Re: [PR] optimize: RM TM startup connect server fail fast [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on code in PR #6004: URL: https://github.com/apache/incubator-seata/pull/6004#discussion_r1441515875 ## core/src/test/java/io/seata/core/rpc/netty/TmNettyClientTest.java: ## @@ -123,6 +135,25 @@ public void setApplicationId() throws Exception { } +

Re: [PR] optimize: RM TM startup connect server fail fast [incubator-seata]

2024-01-04 Thread via GitHub
funky-eyes commented on code in PR #6004: URL: https://github.com/apache/incubator-seata/pull/6004#discussion_r1441514668 ## changes/en-us/2.x.md: ## @@ -39,13 +39,15 @@ Add changes here for all PR submitted to the 2.x branch. - [[#6184](https://github.com/apache/incubator-seat

[I] seata deploy workflow failed. [incubator-seata-website]

2024-01-04 Thread via GitHub
slievrly opened a new issue, #745: URL: https://github.com/apache/incubator-seata-website/issues/745 ![image](https://github.com/apache/incubator-seata-website/assets/8758457/63233af1-7103-4731-b4d5-eb44b838e5df) https://github.com/apache/incubator-seata-website/actions/runs/7342913539/j