Hi Haibo,

Thanks for the continued feedback. Attached is v8 addressing your nits
on patch 2:

- Reworded the commit message to say "the same outer-bounds-based
  estimator can be reused" instead of implying the code is just
  duplicated.
- Made comments in multirangejoinsel type-neutral where they
  unnecessarily said "range" (e.g. "bound histograms" instead of
  "range histograms", "empty values" instead of "empty ranges").
- Added the reverse mixed-direction tests (multirange x range).

Regards,
Maxime

Attachment: v8-0001-Improve-range-join-selectivity-estimation-for.patch
Description: v8-0001-Improve-range-join-selectivity-estimation-for.patch

Attachment: v8-0002-Improve-multirange-join-selectivity-estimation-fo.patch
Description: v8-0002-Improve-multirange-join-selectivity-estimation-fo.patch

Attachment: v8-0003-Remove-duplicate-selectivity-functions-between-ra.patch
Description: v8-0003-Remove-duplicate-selectivity-functions-between-ra.patch

Reply via email to