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

Increase redshift path inputs to use max varchar instead of default (256) #26

Closed
rlh1994 opened this issue Aug 14, 2023 · 0 comments · Fixed by #30
Closed

Increase redshift path inputs to use max varchar instead of default (256) #26

rlh1994 opened this issue Aug 14, 2023 · 0 comments · Fixed by #30
Labels
category:macros Related to the macros in the package. priority:medium On the roadmap. type:enhancement New features or improvements to existing features.

Comments

@rlh1994
Copy link
Member

rlh1994 commented Aug 14, 2023

Is your feature request related to a problem? Please describe.

Some users have a long path, well over the default 256 that redshift uses when only varchar is provided. They then need to overwrite the macro themselves but this gets overwritten again when updating the package.

Describe the solution you'd like

Use varchar(65535) instead of varchar in the redshift udf macros.

Describe alternatives you've considered

Additional context

Are you interested in contributing this feature?
@rlh1994 rlh1994 added type:enhancement New features or improvements to existing features. category:macros Related to the macros in the package. labels Aug 14, 2023
@github-actions github-actions bot added the status:needs_triage Needs maintainer triage. label Aug 14, 2023
@rlh1994 rlh1994 added priority:medium On the roadmap. and removed status:needs_triage Needs maintainer triage. labels Aug 14, 2023
agnessnowplow added a commit that referenced this issue Aug 14, 2023
@agnessnowplow
Increase redshift path inputs to use max varchar instead of default 2…
7620dc9 
…56 (Close #26)
@agnessnowplow agnessnowplow mentioned this issue Aug 14, 2023
Merged
4 tasks
agnessnowplow added a commit that referenced this issue Aug 16, 2023
@agnessnowplow
Increase redshift path inputs to use max varchar instead of default 2…
3885fb9 
…56 (Close #26)
agnessnowplow added a commit that referenced this issue Aug 16, 2023
@agnessnowplow
Increase redshift path inputs to use max varchar instead of default 2…
0b59355 
…56 (Close #26)
@agnessnowplow agnessnowplow mentioned this issue Aug 16, 2023
Merged
6 tasks
agnessnowplow added a commit that referenced this issue Aug 17, 2023
@agnessnowplow
Increase redshift path inputs to use max varchar instead of default 2…
8045d95 
…56 (Close #26)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
category:macros Related to the macros in the package. priority:medium On the roadmap. type:enhancement New features or improvements to existing features.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Release/snowplow fractribution/0.3.4 snowplow/dbt-snowplow-fractribution
1 participant
@rlh1994