Skip to content
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

fix: move big message ut from message begin to message nocopy #528

Merged
merged 1 commit into from
Jul 12, 2022

Conversation

Lvnszn
Copy link
Contributor

@Lvnszn Lvnszn commented Jul 1, 2022

What type of PR is this?

test

What this PR does / why we need it (en: English/zh: Chinese):

en: big message won't happened in message begin, move to message no copy
zh: 大的消息体不会存在 message begin 这个函数里面,所以迁移到 message no copy 这个 ut 里面

Which issue(s) this PR fixes:

@Lvnszn Lvnszn force-pushed the fix/move_ut branch 3 times, most recently from a3af97a to 129863d Compare July 1, 2022 10:05
@Lvnszn Lvnszn force-pushed the fix/move_ut branch 3 times, most recently from e04e548 to c18fd58 Compare July 1, 2022 14:33
@joway joway merged commit 8169eb5 into cloudwego:develop Jul 12, 2022
@Lvnszn Lvnszn deleted the fix/move_ut branch July 12, 2022 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants