velox icon indicating copy to clipboard operation
velox copied to clipboard
verified publisher icon facebookincubator

feat(SelectivityIterator): Enable copy constructor for SelectivityIterator

Open akoshelev opened this issue 7 months ago • 3 comments

Summary: I am trying to write a UDF function that will operate on std::range of input values to avoid copying them to a new container.

It works if I makeSelectivityIterator copyable as ranges require that. I still need to build a wrapper around it to add iterator tags and iterator concepts, but it is not too bad as long as iterator logic is encapsulated in SelectivityIterator.

Differential Revision: D81847304

akoshelev avatar Sep 08 '25 05:09 akoshelev

Deploy Preview for meta-velox canceled.

Name Link
Latest commit 87552530aefc75aa427fb692a45ccedde58679cb
Latest deploy log https://app.netlify.com/projects/meta-velox/deploys/68be649aa804c60009492b91

netlify[bot] avatar Sep 08 '25 05:09 netlify[bot]

This pull request was exported from Phabricator. Differential Revision: D81847304

facebook-github-bot avatar Sep 08 '25 05:09 facebook-github-bot

This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the PR, make sure you've addressed reviewer comments, and rebase on the latest main. Thank you for your contributions!

stale[bot] avatar Dec 17 '25 08:12 stale[bot]