apache / datafusion-ballista

Apache DataFusion Ballista Distributed Query Engine

https://datafusion.apache.org/ballista

Apache License 2.0

1.56k stars 197 forks source link

Refactor lru mod, remove linked_hash_map #918

Closed PsiACE closed 1 year ago

PsiACE commented 1 year ago

Which issue does this PR close?

Closes #.

Rationale for this change

I noticed that in order to implement cache, we have to maintain a LinkedHashMap derived from hashlink. Now that hashlink can work with hashbrown 0.14, it is time to remove it.

What changes are included in this PR?

remove LinkedHashMap, work with hashlink
reorg mods

Are there any user-facing changes?

PsiACE commented 1 year ago

Hi @Dandandan, @yahoNanJing , @r4ntix, could you help review this PR?