On Mon, Nov 15, 2021 at 2:02 PM Michael McCandless <[email protected]> wrote: > > > Yeah I love that idea, but that's not what Patrick's PR explored (yet?). > > His explored switching away from custom token positions to NumericDocValues > to store the same data (ordinal -> parent mapping), but it still loaded all > of those into massive heap-resident int[]. > > I agree it would be awesome to try avoiding those big int[] and reading live > from NumericDocValues during faceting! It would require some re-work of the > facetting code to e.g. sort the ordinals to (efficiently) visiting them in > forward iterator-friendly order. > > But that is a different change and probably we should not hold 9.0 for it? >
Agreed: I was confused about the scope of the change. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
