Add ability to read Thrift SnowplowRawEvent to common-enrich

[alexanderdean]

Assigning to Brandon but I'm (@alexanderdean) very happy to help!

[alexanderdean]

Depends on #475

[bamos]

Hi @alexanderdean - should we rename one of com.snowplowanalytics.snowplow.collectors.thrift.TrackerPayload and com.snowplowanalytics.snowplow.enrich.common.inputs.TrackerPayload?

[alexanderdean]

Hmm, good point. Of the two of these, thrift.TrackerPayload is the more permanent, so I guess we can import inputs.TrackerPayload => ITP or similar?

[bamos]

Okay, I'll do that

[bamos]

Hm, with renaming, I get importedTrackerPayload' is permanently hidden by definition of object TrackerPayload in package inputs, so I'm doing thrift.TrackerPayload => ThriftTrackerPayload.

[alexanderdean]

thrift.TrackerPayload => ThriftTrackerPayload sounds good to me...

[bamos]

Hi @alexanderdean - just taking a shot here. Can you make a pass when you get a chance?

[alexanderdean]

For sure - will do!

[alexanderdean]

Assigning to Alex to review.

In the meantime, have published current as a snapshot:

info] Main Scala API documentation successful.
[info] Packaging /vagrant/snowplow/3-enrich/scala-common-enrich/target/snowplow-common-enrich-0.2.0-SNAPSHOT-javadoc.jar ...
[info] Done packaging.
[info]  published snowplow-common-enrich to /var/www/maven.snplow.com/prod/public/snapshots/com/snowplowanalytics/snowplow-common-enrich/0.2.0-SNAPSHOT/snowplow-common-enrich-0.2.0-SNAPSHOT.pom
[info]  published snowplow-common-enrich to /var/www/maven.snplow.com/prod/public/snapshots/com/snowplowanalytics/snowplow-common-enrich/0.2.0-SNAPSHOT/snowplow-common-enrich-0.2.0-SNAPSHOT.jar
[info]  published snowplow-common-enrich to /var/www/maven.snplow.com/prod/public/snapshots/com/snowplowanalytics/snowplow-common-enrich/0.2.0-SNAPSHOT/snowplow-common-enrich-0.2.0-SNAPSHOT-sources.jar
[info]  published snowplow-common-enrich to /var/www/maven.snplow.com/prod/public/snapshots/com/snowplowanalytics/snowplow-common-enrich/0.2.0-SNAPSHOT/snowplow-common-enrich-0.2.0-SNAPSHOT-javadoc.jar
[success] Total time: 26 s, completed Jan 9, 2014 4:42:26 PM

[bamos]

Awesome, thanks @alexanderdean!

[alexanderdean]

This is looking great. I will add a few more tests when I get a chance but let's treat this ticket as complete now.