velox icon indicating copy to clipboard operation
velox copied to clipboard

Limit Dictionary to Single Level

Open oerling opened this issue 1 year ago • 4 comments

  • Modifies wrapInDictionary to flatten indices of a wrapped dictionary iwth the wrapping indices.
  • Makes lazy loading of a dictionary encoded column to combine the indices with a dictionary wrapper if loading with lazy wrapped in a dictionary.
  • Adds functions to transpose dictionaries with and without nulls.
  • Changes NestedLoopJoin and MergeJoin so that they wrap their input only after the wrapping indices are known. Previously these would wrap first and only then fill in the indices.
  • Checks that we do not come across multiple nested dictionaries.

oerling avatar Sep 06 '24 18:09 oerling

Deploy Preview for meta-velox canceled.

Name Link
Latest commit d121f543d7c7a2a7a3f8405a68c1555f02df376b
Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/676362d50e32ee0008cca9b1

netlify[bot] avatar Sep 06 '24 18:09 netlify[bot]

@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Sep 06 '24 18:09 facebook-github-bot

@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Oct 22 '24 00:10 facebook-github-bot

@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Oct 23 '24 23:10 facebook-github-bot

@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Nov 06 '24 23:11 facebook-github-bot

@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Dec 13 '24 20:12 facebook-github-bot

@oerling has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Dec 19 '24 00:12 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 Mar 19 '25 09:03 stale[bot]