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

[FEA] Support parse_url #6969

Closed
viadea opened this issue Nov 1, 2022 · 5 comments
Closed

[FEA] Support parse_url #6969

viadea opened this issue Nov 1, 2022 · 5 comments
Labels
feature request New feature or request

Comments

@viadea
Copy link
Collaborator

viadea commented Nov 1, 2022

I wish we can support parse_url function.

eg:

spark-sql> select parse_url(c_customer_id,'HOST') from tpcds.customer limit 10;

      ! <ParseUrl> parse_url(c_customer_id#1, HOST, false) cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.catalyst.expressions.ParseUrl
@viadea viadea added feature request New feature or request ? - Needs Triage Need team to review and classify labels Nov 1, 2022
@sameerz
Copy link
Collaborator

sameerz commented Nov 2, 2022

Discuss with the python cudf team whether they could benefit from a kernel for this functionality in libcudf.

@mattahrens
Copy link
Collaborator

Is it sufficient for the initial implementation that the 2nd and 3rd parameters are literals?

@viadea
Copy link
Collaborator Author

viadea commented Apr 20, 2023

From the logs, i think it is good enough. I am also double checking with user.

@viadea
Copy link
Collaborator Author

viadea commented Nov 1, 2023

The requirement is to support all 3 input parameter for this function including the KEY.

We need to support PATH, QUERY and HOST.

@hyperbolic2346
Copy link
Collaborator

Parse_url is now supported for Host with this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants