Channels
announcements
ask-marvin
best-practices-coordination-plane
data-ecosystem
data-tricks-and-tips
events
find-a-prefect-job
geo-bay-area
geo-berlin
geo-boston
geo-chicago
geo-colorado
geo-dc
geo-israel
geo-japan
geo-london
geo-nyc
geo-seattle
geo-texas
gratitude
introductions
marvin-in-the-wild
pacc-clearcover-june-12-2023
pacc-may-31-2023
ppcc-may-16-2023
prefect-ai
prefect-aws
prefect-azure
prefect-cloud
prefect-community
prefect-contributors
prefect-dbt
prefect-docker
prefect-gcp
prefect-getting-started
prefect-integrations
prefect-kubernetes
prefect-recipes
prefect-server
prefect-ui
random
show-us-what-you-got
Title
b
Brian Mesick
10/25/2019, 5:44 PMHi folks. I'm trying out Prefect and running some pretty basic flows to get a feel for how development works. I'm curious if anyone is actually using the Snowflake task as it seems to have a bug that I would expect to render it unusable.
https://github.com/PrefectHQ/prefect/blob/master/src/prefect/tasks/snowflake/snowflake.py#L101 is calling
executequeryChanging that line to
executed = cursor.execute(query, params=data)c
Chris White
10/25/2019, 5:50 PMHi Brian, this task was contributed by a user who presumably was using it; that being said, I see the bug and we’d definitely appreciate a PR which fixes it!
b
Brian Mesick
10/25/2019, 5:51 PMSure thing, just wanted to make sure I wasn't missing something obvious.
👍 1
j
Jeremiah
10/25/2019, 5:54 PMIt looks like the Python Snowflake library had a 2.0 release a couple weeks ago - this could be a breaking change it introduced
b
Brian Mesick
10/25/2019, 5:55 PMThis signature doesn't seem to have changed going back to at least 1.3.8
m
Mark McDonald
10/26/2019, 7:29 PMlooks like the arguments name should be
commandqueryI threw up a PR for it: https://github.com/PrefectHQ/prefect/pull/1663
💯 2