Closed mattmonihan closed 8 years ago
@mattmonihan i'm having a hard time connecting to redshift from home (dunno why i've done it fine in the recent past) so i'll have to actually run these at some later point. for now, just have those two comments.
@jthandy Did you VPN to the office?
@mattmonihan yeah idunno what was wrong I restarted and it worked. but pipeline hasn't pulled over the magento tables yet...annoying.
Ok. Validated this against the new schema. With the two new commits I'm good.
:+1:
Great stuff; excited to see more coalesce around magento. It's obviously a complicated data model and has a ton of users.
(merging)
I'm not sure what the rubric for these PRs is, but I thought I'd put this out there for comment.
I haven't been able to confirm that the query runs since the "magento" schema isn't yet available in my redshift instance. (The DB is hooked up to our pipeline instance, however.) We should consider that lag time since I have to wait test, disrupting my workflow.
The analysis query should produce a table that looks like the following: