Skip to content

Commit

Permalink
Rename reserved word column in snowflake quickstart (#20344)
Browse files Browse the repository at this point in the history
Snowflake serialization failed for me because of the reserved keyword
"by"

The SnowflakeIOManager does not quote identifiers (apparently on
purpose)

## Summary & Motivation
Out of the box, a cloned copy of this quickstart failed for me on
Dagster cloud.

## How I Tested These Changes
I ran the updated quickstart on dagster cloud.
  • Loading branch information
jcampbell authored Mar 8, 2024
1 parent 3cdb48c commit f7d2611
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ def hackernews_topstories(

df = pd.DataFrame(results)

# Rename the column to avoid conflict with the reserved keyword "by"
df.rename(columns={"by": "by_"}, inplace=True)

# Dagster supports attaching arbitrary metadata to asset materializations. This metadata will be
# shown in the run logs and also be displayed on the "Activity" tab of the "Asset Details" page in the UI.
# This metadata would be useful for monitoring and maintaining the asset as you iterate.
Expand Down

0 comments on commit f7d2611

Please sign in to comment.