Repository navigation
[Filestore] Use RcBufAllocator - #7499
antonmyagkov wants to merge 1 commit into
Conversation
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo.
|
ecd1a5a to
6ed1235
Compare
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo. 🔴 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 5460s) Test run completed, no test results found for commit 6ed1235. 🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 5165s): all tests PASSED for commit 6ed1235.
|
6ed1235 to
7f9e508
Compare
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo.
|
7f9e508 to
5c721b7
Compare
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo.
|
5c721b7 to
e8e32b9
Compare
e8e32b9 to
c01b38d
Compare
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo. 🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 5017s): all tests PASSED for commit c01b38d.
|
| // to remove unnecessary memcpy | ||
| // GetRcBufAllocator/GetDefaultRcBufAllocator is available | ||
| // starting from ydb/stable-26-1 | ||
| #ifdef OPENSOURCE |
There was a problem hiding this comment.
Let’s add a TODO with a link to the issue tracking the removal of this workaround
Using RcBufAllocator is required to pass buffer via RDMA(if it's enabled in IC config)