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

Warning when parsing filepath in pandas.SQLQueryDataset even though pipeline works fine. #1797

Closed
galenseilis opened this issue Mar 8, 2024 · 0 comments · Fixed by #1843
Closed
Assignees
Labels
Issue: Bug Report Quick Win Low/Medium priorities but quick to do

Comments

@galenseilis
Copy link

Description

https://kedro-org.slack.com/archives/C03RKP2LW64/p1709912797979079

Context

How has this bug affected you? What were you trying to accomplish?

The impact is relatively minor. I just see a warning.

Steps to Reproduce

  1. Put a SQL script into an accessible path.
  2. Put a SQL query dataset in the catalog with a filepath assigned to the path of the SQL script.
  3. Run kedro.

Expected Result

I expect that a warning should not be raised if nothing is wrong or situationally risky.

Actual Result

I get a warning even though the pipeline works as desired. Purportedly this might be a Kedro-viz issue.

See here.

Your Environment

Include as many relevant details as possible about the environment you experienced the bug in:

  • Web browser system and version: N/A
  • Operating system and version: Windows 10
  • NodeJS version used (if relevant): N/A
  • Kedro version used (if relevant): See here.
  • Python version used (if relevant): 3.10
@ravi-kumar-pilla ravi-kumar-pilla self-assigned this Mar 8, 2024
@ravi-kumar-pilla ravi-kumar-pilla moved this to Inbox in Kedro-Viz Mar 8, 2024
@rashidakanchwala rashidakanchwala added the Quick Win Low/Medium priorities but quick to do label Mar 11, 2024
@ravi-kumar-pilla ravi-kumar-pilla moved this from Inbox to Backlog in Kedro-Viz Apr 2, 2024
@rashidakanchwala rashidakanchwala moved this from Backlog to Todo in Kedro-Viz Apr 2, 2024
@ravi-kumar-pilla ravi-kumar-pilla moved this from Todo to In Progress in Kedro-Viz Apr 2, 2024
@ravi-kumar-pilla ravi-kumar-pilla moved this from In Progress to In Review in Kedro-Viz Apr 5, 2024
@github-project-automation github-project-automation bot moved this from In Review to Done in Kedro-Viz Apr 8, 2024
@ravi-kumar-pilla ravi-kumar-pilla mentioned this issue Apr 8, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Bug Report Quick Win Low/Medium priorities but quick to do
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants