Allen Madsen
06/10/2025, 9:39 PMGian Merlino
06/17/2025, 6:34 AMapplyAll never called, it also seems like it's never overridden. It might have been an idea that never quite saw the light of day
Probably the easiest code path to make use it would be the LookupExprMacro vectorized path. Being vectorized, it's already set up to deal in batches. Currently it calls apply but that could change to applyAll.
The join code path is more difficult because it isn't already set up to deal in batches. But it could possibly be done with some kind of lookahead.
Btw, IMO, the signature of applyAll isn't ideal. In returning a Map it sort of forces a less efficient double-Map-lookup path for lookups backed by fully in-memory Maps. It'd probably be better for it to take a list of keys and return a list of values in the same orderAllen Madsen
06/17/2025, 8:07 PMAllen Madsen
06/17/2025, 8:11 PMBtw, IMO, the signature ofisn't ideal. In returning aapplyAllit sort of forces a less efficient double-Map-lookup path for lookups backed by fully in-memory Maps. It'd probably be better for it to take a list of keys and return a list of values in the same orderMap
Allen Madsen
06/17/2025, 8:11 PM