Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(datafusion): add map methods to datafusion compiler #10510

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

venkat-oss
Copy link

@venkat-oss venkat-oss commented Nov 19, 2024

  • Added Datafusion supported map functions to Datafusion compiler.
  • This PR in Datafusion repo adds support for certain map functions.

ops.Map - ✅ (Already supported by ibis)
ops.MapLength - ✅
ops.MapGet - ✅
ops.MapContains - ✅
ops.MapKeys - ✅
ops.MapValues - ✅
ops.MapMerge - 🆘 Need help

@github-actions github-actions bot added the sql Backends that generate SQL label Nov 19, 2024
@venkat-oss venkat-oss changed the title feat: add datafusion map methods to datafusion compiler feat(datafusion): add datafusion map methods to datafusion compiler Nov 19, 2024
@venkat-oss venkat-oss changed the title feat(datafusion): add datafusion map methods to datafusion compiler feat(datafusion): add map methods to datafusion compiler Nov 19, 2024
@venkat-oss
Copy link
Author

venkat-oss commented Nov 19, 2024

@gforsyth and @cpcloud could you please provide some feedback/pointers on how MapMerge could be implemented?

I'm not very sure how MapMerge could be implemented unless supported by datafusion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sql Backends that generate SQL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant