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

Add state to run manifests #29

Open
chuwy opened this issue Jun 8, 2017 · 5 comments
Open

Add state to run manifests #29

chuwy opened this issue Jun 8, 2017 · 5 comments

Comments

@chuwy
Copy link
Contributor

chuwy commented Jun 8, 2017

Currently we have only RunId key in run manifest table - enriched-archive/run=2017-05-01-12-00-00. It would be useful to add State with STARTED and COMPLETED to see what folders were failed.
@alexanderdean
Copy link
Member

SUCCEEDED not COMPLETED for clarity...

@chuwy
Copy link
Contributor Author

chuwy commented Jun 8, 2017 •
edited
Loading

I actually think instead of State key it should be separate Started key with date and Suceeded with date/null. This way we can get additional information on when folders were processed.

Edit: Suceeded

@alexanderdean
Copy link
Member

StartedAt, SucceededAt - I think that works...

@chuwy chuwy mentioned this issue Jun 8, 2017
Open
@chuwy
Copy link
Contributor Author

chuwy commented Jun 8, 2017

@alexanderdean do you think this feature can go outside Analytics SDK? Like maintaining state LoadedAt from RDB Loader or AddedAt from EmrEtlRunner (or whatever we'll be responsible in that moment).

@alexanderdean
Copy link
Member

I am not sure if this answers your question @chuwy, but I 100% envisage the Scala Analytics SDK being used in e.g. the RDB Loader, and thus yes using this state tracking...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alexanderdean @chuwy