yahoo / Oak

A Scalable Concurrent Key-Value Map for Big Data Analytics
Apache License 2.0
268 stars 48 forks source link

Off-heap (direct) entry array #186

Closed liran-funaro closed 2 years ago

liran-funaro commented 2 years ago

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

liran-funaro commented 2 years ago

@sanastas

we have no good separation between Hash logic and Map logic

I think it is possible to attain this separation with not too much effort. If you find methods that should be moved like the ones you already mentioned, then let me know. Anyway, I won't start applying the changes you requested and merging with the master until we feel confident that we want to go ahead with this change.

sanastas commented 2 years ago

I want to ask @boris-kap to make the review to this PR as well, so we have more then one opinion.

liran-funaro commented 2 years ago

Broke down into number of PRs and rethink chunk/array design