apache / apache/accumulo

Update Native Maps to use more recent C++ standards

Open
#1,386 21 comments 0 reactions 1 assignee Claimed by @phrocker View on GitHub
enhancement
Dominant language
Java
Stars
1.2k
Forks
487
Avg merge
4d 5h
Merged PRs (30d)
13

Description

It may also help us to move away from raw pointers for internal objects where we can use RAII to better manage resources. Only top level objects need a java managed lifetime. everything else should use smart pointers. this alleviates any potential concern for memory leaks and also allow us to better manage object lifetimes.

Further, I think we can better exploit move semantics and improve performance. As a final task in this, we should look at copy semantics where they are employed. It seems that memory is copied quite a bit. In some cases it is not avoidable, but it would behoove us to evaluate their usages.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.