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

Scala Kinesis Enrich: etl_tstamp should be Redshift Formatted not raw #1842

Closed
jbeemster opened this issue Jul 2, 2015 · 2 comments
Closed
Assignees

Comments

@jbeemster jbeemster self-assigned this Jul 2, 2015
@jbeemster jbeemster added this to the Release 67 Bohemian Waxwing milestone Jul 2, 2015
@alexanderdean alexanderdean changed the title Scala Kinesis Enrich: etl_tstamp should be Redshift Formatted not Raw Scala Kinesis Enrich: etl_tstamp should be Redshift Formatted not raw Jul 2, 2015
@denismo
Copy link

denismo commented Jul 6, 2015

Also, would like to throw in that the timestamp (originally) was in local time, not UTC. Not sure whether the current fix handles that.

@jbeemster
Copy link
Member Author

Hey @denismo yes this does indeed fix not UTC as well! Instead of it being a raw timestamp it is now being processed using this formatter: https://github.com/snowplow/snowplow/blob/master/3-enrich/scala-common-enrich/src/main/scala/com.snowplowanalytics.snowplow.enrich/common/enrichments/EventEnrichments.scala#L39

So it should always be redshift compatible and in UTC from now on.

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