apache / incubator-seata

:fire: Seata is an easy-to-use, high-performance, open source distributed transaction solution.
https://seata.apache.org/
Apache License 2.0
25.3k stars 8.78k forks source link

AbstractSessionManager#updateBranchSessionStatus debug log is wrong #1538

Closed TinyZzh closed 5 years ago

TinyZzh commented 5 years ago

Ⅰ. Issue Description

image

日志输出的内容是否应该分别修改为BRANCH_UPDATE和BRANCH_REMOVE?

Ⅱ. Describe what happened

If there is an exception, please attach the exception trace:

Just paste your stack trace here!

Ⅲ. Describe what you expected to happen

Ⅳ. How to reproduce it (as minimally and precisely as possible)

  1. xxx
  2. xxx
  3. xxx

Ⅴ. Anything else we need to know?

Ⅵ. Environment:

fescar-robot commented 5 years ago

Hi @TinyZzh, we detect non-English characters in the issue. This comment is an auto translation from @fescar-robot to help other users to understand this issue. We encourage you to describe your issue in English which is more friendly to other users.

AbstractSessionManager#updateBranchSessionStatus debug log is wrong

Ⅰ. Issue Description

image

Should the contents of the log output be modified to BRANCH_UPDATE and BRANCH_REMOVE respectively?

Ⅱ. Describe what happened

If there is an exception, please attach the exception trace:

Just paste your stack trace here!

Ⅲ. Describe what you expected to happen

Ⅳ. How to reproduce it (as minimally and precisely as possible)

  1. xxx
  2. xxx
  3. xxx

Ⅴ. Anything else we need to know?

Ⅵ. Environment:

zjinlei commented 5 years ago

thanks I will fix it