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(rust, python): allow expressions as arguments in str.ends_with #6361

Merged
merged 4 commits into from
Jan 23, 2023

Conversation

gab23r
Copy link
Contributor

@gab23r gab23r commented Jan 21, 2023

Same as startswith

@@ -56,8 +56,7 @@ fn test_q2() -> PolarsResult<()> {
.inner_join(nation(), "s_nationkey", "n_nationkey")
.inner_join(region(), "n_regionkey", "r_regionkey")
.filter(col("p_size").eq(15))
.filter(col("p_type").str().ends_with("BRASS"));

.filter(col("p_type").str().ends_with("BRASS".into()));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be lit("BRASS".into::<String>())

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks,
This test does not run locally, do you know why ? make test is not enough ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is in the integration tests. Can you run those?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, it is because I was running testing only in polars/py-polars, and I should run them in polars/polars right ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't make .into::<String>()) to work, but I think it works with .to_string()

@gab23r
Copy link
Contributor Author

gab23r commented Jan 22, 2023

By the way, when I ran the integration test, I got 153 failed run. My problem looks really similar to this one #3121 ( even if I am on Ubuntu). Is there a way to solve this ?

@gab23r gab23r changed the title allow expressions as arguments in str.ends_with feat(rust, python): allow expressions as arguments in str.ends_with Jan 22, 2023
@github-actions github-actions bot added enhancement New feature or an improvement of an existing feature python Related to Python Polars rust Related to Rust Polars labels Jan 22, 2023
@ritchie46
Copy link
Member

By the way, when I ran the integration test, I got 153 failed run. My problem looks really similar to this one #3121 ( even if I am on Ubuntu). Is there a way to solve this ?

Did you take a look at our CI? That shows all setup steps.

@ritchie46
Copy link
Member

Thanks @gab23r!

@ritchie46 ritchie46 merged commit 294f6c5 into pola-rs:master Jan 23, 2023
@gab23r gab23r deleted the endswith_expr branch February 13, 2023 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or an improvement of an existing feature python Related to Python Polars rust Related to Rust Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants