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

[Core feature] Add LazyFrame support for polars plugin #5678

Closed
2 tasks done
cosmicBboy opened this issue Aug 20, 2024 · 1 comment · Fixed by flyteorg/flytekit#2695
Closed
2 tasks done

[Core feature] Add LazyFrame support for polars plugin #5678

cosmicBboy opened this issue Aug 20, 2024 · 1 comment · Fixed by flyteorg/flytekit#2695
Assignees
Labels
enhancement New feature or request

Comments

@cosmicBboy
Copy link
Contributor

Motivation: Why do you think this is important?

Currently, the flytekit polars plugin only supports polars.DataFrame, but many polars users also use LazyFrames.

Goal: What should the final outcome look like, ideally?

The flytekit type transformers should support pl.LazyFrame annotation and StructuredDataset encoding/decoding

Describe alternatives you've considered

Today users can convert LazyFrames to DataFrame with a collect() call.

Propose: Link/Inline OR Additional context

No response

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes
@cosmicBboy cosmicBboy added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels Aug 20, 2024
Copy link

runllm bot commented Aug 20, 2024

Currently, the Flytekit Polars plugin supports polars.DataFrame but not LazyFrame (source). Users can convert LazyFrame to DataFrame using the collect() method as a workaround.

Would you like more details on how to extend Flytekit to support LazyFrame?

Sources:


This is an AI-generated response and your feedback is appreciated! Please leave a 👍 if this is helpful and 👎 if it is not.

Mention @RunLLM to ask follow-up questions. See runllm.com for more!

@cosmicBboy cosmicBboy self-assigned this Aug 22, 2024
@cosmicBboy cosmicBboy removed the untriaged This issues has not yet been looked at by the Maintainers label Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant