curve icon indicating copy to clipboard operation
curve copied to clipboard

[compile] update brpc to release1.3 version

Open fansehep opened this issue 2 years ago • 13 comments

Signed-off-by: fan [email protected]

What problem does this PR solve?

Issue Number: https://github.com/opencurve/curve/issues/2034#event-7780133596

Problem Summary:

What is changed and how it works?

What's Changed:

How it Works:

Side effects(Breaking backward compatibility? Performance regression?): some unix_socket patch test has some errors but can compile pass.

Check List

  • [ ] Relevant documentation/comments is changed or added
  • [ ] I acknowledge that all my contributions will be made under the project's license

fansehep avatar Nov 17 '22 14:11 fansehep

recheck

ilixiaocui avatar Nov 18 '22 01:11 ilixiaocui

The release1.3 bRPC has many large changes in some places. I just make the new patch easily. But the patch with unix_socket test is failed in echo_c++_unix_socket_server and echo_c++_unix_socket_client client. It seems I do something wrong with patch. Can the old patch author help me to fix it? I don't know more about bRPC.

fansehep avatar Nov 18 '22 04:11 fansehep

The release1.3 bRPC has many large changes in some places. I just make the new patch easily. But the patch with unix_socket test is failed in echo_c++_unix_socket_server and echo_c++_unix_socket_client client. It seems I do something wrong with patch. Can the old patch author help me to fix it? I don't know more about bRPC.

Can you paste the error output?

ilixiaocui avatar Nov 18 '22 06:11 ilixiaocui

The release1.3 bRPC has many large changes in some places. I just make the new patch easily. But the patch with unix_socket test is failed in echo_c++_unix_socket_server and echo_c++_unix_socket_client client. It seems I do something wrong with patch. Can the old patch author help me to fix it? I don't know more about bRPC.

Can you paste the error output?

Just the echo_unix_socket_client can not connect echo_unix_socket_server.

2022-11-18_14-11-1668752337

I conjecture it is my new patch error.

fansehep avatar Nov 18 '22 06:11 fansehep

@ilixiaocui, @wu-hanqing has help me to fix the test error. Now the release-1.3 version bRPC has been support unix_socket. Should we use upstream unix_socket?

fansehep avatar Nov 19 '22 01:11 fansehep

@ilixiaocui @wu-hanqing

wuhongsong avatar Feb 16 '23 07:02 wuhongsong

@ilixiaocui @wuhongsong

Cyber-SiKu avatar Feb 28 '23 02:02 Cyber-SiKu

Not only the domain_unixsocket_test is fail, and many tests.

More test failures can use this patch and run brpc_tests.

fansehep avatar Mar 14 '23 08:03 fansehep

Not only the domain_unixsocket_test is fail, and many tests.

More test failures can use this patch and run brpc_tests.

How about submit the patch to bRPC?

legionxiong avatar Mar 14 '23 08:03 legionxiong

Not only the domain_unixsocket_test is fail, and many tests. More test failures can use this patch and run brpc_tests.

How about submit the patch to bRPC?

I think it is not a good idea. bRPC will not accept our customized patch.

fansehep avatar Mar 14 '23 08:03 fansehep

recheck

fansehep avatar May 29 '23 12:05 fansehep

cicheck

Cyber-SiKu avatar Jun 21 '23 02:06 Cyber-SiKu

@fansehep Keep on going? or any help?

Cyber-SiKu avatar Jun 21 '23 02:06 Cyber-SiKu