k3dom commented on PR #144: URL: https://github.com/apache/datasketches-rust/pull/144#issuecomment-4880028681
> This is proposed by my Codex at the first time. But I reject it because an active_entries iterator would do the activity check every time while we'd iterator it several (now 3) times. So it's not a net win. Yeah agree, that is what I meant when I said it's probably not faster for large maps. For most map sizes the three scans should be cheap but sparse scans probably loose to collect-once. > But Row<'a, T> is possible, anyway. I just need one more consideration and would prefer a real use case. Agree, I don't have a use case for my project right now either, just saw I mention it because I noticed it while going over the API once again -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
