kvrocks icon indicating copy to clipboard operation
kvrocks copied to clipboard

Published 20 hours ago • verified publisher icon apache

Replace mutexes with tbb concurrent containers if proper

Open PragmaTwice opened this issue 1 year ago • 5 comments

Search before asking

  • [X] I had searched in the issues and found no similar issues.

Motivation

With TBB, now we can eliminate some mutexes applied on containers in our codebase with concurrent containers directly.

Solution

No response

Are you willing to submit a PR?

  • [ ] I'm willing to submit a PR!

PragmaTwice avatar Apr 14 '23 07:04 PragmaTwice

tbb or folly::concurrentHashMap ?

wanghenshui avatar Apr 18 '23 06:04 wanghenshui

tbb or folly::concurrentHashMap ?

We currently do not introduce folly as our dependency, since it is a far more large library.

PragmaTwice avatar Apr 18 '23 07:04 PragmaTwice

hey @PragmaTwice I want to try and investigate the tbb containter.

infdahai avatar Jun 12 '23 19:06 infdahai

hey @PragmaTwice I want to try and investigate the tbb containter.

Assigned. Good luck.

PragmaTwice avatar Jun 16 '23 13:06 PragmaTwice

Hi @PragmaTwice ,

This issue seems to be stale for one year. Could this issue be assigned to me if it still needs to be resolved? I want to research the tbb container and make some contribution.

Best Regards, Edward

LindaSummer avatar Jul 08 '24 11:07 LindaSummer