Alluxio / alluxio

Alluxio, data orchestration for analytics and machine learning in the cloud
https://www.alluxio.io
Apache License 2.0
6.87k stars 2.94k forks source link

Support trie implemention for FieldIndex #18673

Closed thu-david closed 1 month ago

thu-david commented 3 months ago

What changes are proposed in this pull request?

I have added new implemention for fieldIndex when use pageid as key.

Why are the changes needed?

Because the hashmap use much memory when save so many same prefix id.

Does this PR introduce any user facing changes?

No.

alluxio-bot commented 2 months ago

Automated checks report:

Some checks failed. Please fix the reported issues and reply alluxio-bot, check this please to re-run checks.

alluxio-bot commented 2 months ago

Automated checks report:

All checks passed!

thu-david commented 1 month ago

I have fixed the conflict.

YichuanSun commented 1 month ago

alluxio-bot, merge this please.

alluxio-bot commented 1 month ago

merge failed: Merge refused because pull request does not have label start with type-

YichuanSun commented 1 month ago

alluxio-bot, merge this please.