Skip to content

GH-5686 LMDB page walking estimator#5688

Draft
hmottestad wants to merge 5 commits intomainfrom
GH-5686-lmdb-page-walking-estimator
Draft

GH-5686 LMDB page walking estimator#5688
hmottestad wants to merge 5 commits intomainfrom
GH-5686-lmdb-page-walking-estimator

Conversation

@hmottestad
Copy link
Contributor

GitHub issue resolved: #

Briefly describe the changes proposed in this PR:


PR Author Checklist (see the contributor guidelines for more details):

  • my pull request is self-contained
  • I've added tests for the changes I made
  • I've applied code formatting (you can use mvn process-resources to format from the command line)
  • I've squashed my commits where necessary
  • every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

@hmottestad
Copy link
Contributor Author

@kenwenzel want to take a look?

Test results are promising, benchmarks too.

Some of the benchmarks do show some regressions, but I'm wondering if maybe it's just that the previous plan was faster specifically because the estimated cardinalities were inaccurate.

@kenwenzel
Copy link
Contributor

@hmottestad Does it make sense to completely port LMDB to Java? I see that your code already mirrors many of LMDB's internals.

@hmottestad hmottestad force-pushed the GH-5686-lmdb-page-walking-estimator branch from 9f3fa4c to d77a7f5 Compare February 19, 2026 20:23
@hmottestad hmottestad force-pushed the GH-5686-lmdb-page-walking-estimator branch from 37ad89b to f613d47 Compare February 22, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants