-
Notifications
You must be signed in to change notification settings - Fork 8.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bugfix:the buffer may cause overflows when sql statement is long #1148
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1148 +/- ##
=============================================
- Coverage 41.97% 41.96% -0.01%
Complexity 1385 1385
=============================================
Files 246 246
Lines 10240 10239 -1
Branches 1343 1343
=============================================
- Hits 4298 4297 -1
Misses 5378 5378
Partials 564 564
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
core/src/main/java/io/seata/core/protocol/MergedWarpMessage.java
Outdated
Show resolved
Hide resolved
core/src/main/java/io/seata/core/protocol/MergedWarpMessage.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ⅰ. Describe what this PR did
Due to the
java.nio.ByteBuffer
can not grow dynamically, its size is hardcoded now, this patch replacesjava.nio.ByteBuffer
withio.netty.buffer.ByteBuf
, which can grow on demand.Ⅱ. Does this pull request fix one issue?
fixes issue #976
Ⅲ. Why don't you add test cases (unit test/integration test)?
There is already tests upon this method.
Ⅳ. Describe how to verify it
CI passes
Ⅴ. Special notes for reviews
null