Skip to content
This repository has been archived by the owner on Jan 15, 2022. It is now read-only.

enhance function getLastSuccessfulProcessRecord to include process file ... #125

Merged

Conversation

vrushalic
Copy link
Collaborator

...substring (processing dir) instead of null

@vrushalic
Copy link
Collaborator Author

There is an existing function getLastSuccessfulProcessRecord(String cluster), this pull request overloads it to enable passing in the processFileSubstring as well.

@coveralls
Copy link

Coverage Status

Coverage decreased (-5.42%) when pulling 6a0c07e on vrushalic:include_process_filesubstring into 748cd90 on twitter:twitter_only.

@coveralls
Copy link

Coverage Status

Coverage decreased (-5.42%) when pulling 6a0c07e on vrushalic:include_process_filesubstring into 748cd90 on twitter:twitter_only.

@sjlee
Copy link
Collaborator

sjlee commented Dec 13, 2014

LGTM.

jrottinghuis added a commit that referenced this pull request Dec 15, 2014
enhance function getLastSuccessfulProcessRecord to include process file ...
@jrottinghuis jrottinghuis merged commit 87949b6 into twitter:twitter_only Dec 15, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants