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

Redshift: add Redshift DDL for PerformanceTiming #1032

Closed
fblundun opened this issue Oct 6, 2014 · 4 comments
Closed

Redshift: add Redshift DDL for PerformanceTiming #1032

fblundun opened this issue Oct 6, 2014 · 4 comments
Assignees

Comments

@fblundun
Copy link
Contributor

fblundun commented Oct 6, 2014

No description provided.

@fblundun fblundun self-assigned this Oct 6, 2014
@fblundun fblundun added this to the Redshift DDL for JS 2.1.0 milestone Oct 6, 2014
@fblundun
Copy link
Contributor Author

fblundun commented Oct 6, 2014

@deano according to http://docs.aws.amazon.com/redshift/latest/dg/r_names.html Redshift table names are case insensitive. Will this cause any problems with the table name org_w3_PerformanceTiming_1?

@alexanderdean
Copy link
Member

Oh sorry Fred - I neglected to mention that we have an algo for snake_casing the Redshift table names from the JSON Schemas. See for an example:

https://github.com/snowplow/snowplow/blob/master/4-storage/redshift-storage/sql/org.schema/web_page_1.sql

Does that make sense?

@fblundun
Copy link
Contributor Author

fblundun commented Oct 6, 2014

Yes it does! I'll change the table name to org_w3_performance_timing_1.

@alexanderdean
Copy link
Member

Exactly - many thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants