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.

Sign up for GitHub

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

Jump to bottom

[BUG] Sync-up the support of parse_url in qualification tool #1190

Closed
amahussein opened this issue Jul 15, 2024 · 0 comments · Fixed by #1195
Closed

[BUG] Sync-up the support of parse_url in qualification tool #1190

amahussein opened this issue Jul 15, 2024 · 0 comments · Fixed by #1195
Assignees
@amahussein
Labels
bug Something isn't working core_tools Scope the core module (scala)

Comments

@amahussein
Copy link
Collaborator

Describe the bug

By the time we implemented the support of parse_url in Q tool (issue #706), there was a limitation in the GPU support.
For example parse_url (QUERY) was not supported.

Since then, the RAPIDS Plugin added full support for missing arguments:

See the details here:
NVIDIA/spark-rapids#8963
@amahussein amahussein added bug Something isn't working ? - Needs Triage core_tools Scope the core module (scala) labels Jul 15, 2024
@amahussein amahussein mentioned this issue Jul 16, 2024
Closed
@amahussein amahussein self-assigned this Jul 16, 2024
amahussein added a commit to amahussein/spark-rapids-tools that referenced this issue Jul 16, 2024
@amahussein
Sync-up the support of parse_url in qualification tool
be92ed6 
Signed-off-by: Ahmed Hussein (amahussein) <a@ahussein.me>

Fixes NVIDIA#1190

- parse_url(*,query,*) should be treated as supported
- fix the unit-test
@amahussein amahussein mentioned this issue Jul 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amahussein amahussein

Labels
bug Something isn't working core_tools Scope the core module (scala)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Sync-up the support of parse_url in qualification tool amahussein/spark-rapids-tools
1 participant
@amahussein