databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.85k stars 750 forks source link

chore: use dashmap replace hashmap #16426

Closed zhyass closed 2 months ago

zhyass commented 2 months ago

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Use DashMap replace the Mutex<HashMap> and RwLock<HashMap> to make the code more clearly.

DashMap is an implementation of a concurrent associative array/hashmap in Rust.

Tests

Type of change


This change isโ€‚Reviewable

github-actions[bot] commented 2 months ago

Docker Image for PR

note: this image tag is only available for internal use, please check the internal doc for more details.

github-actions[bot] commented 2 months ago

ClickBench Report

github-actions[bot] commented 2 months ago

Docker Image for PR

note: this image tag is only available for internal use, please check the internal doc for more details.

github-actions[bot] commented 2 months ago

ClickBench Report

zhang2014 commented 2 months ago

Recommended to replace only the hashmap in the lock manager.