Skip to content
This repository has been archived by the owner on Apr 4, 2021. It is now read-only.

FALCON-2125 Feed replication fails due to failed post processing acti… #274

Closed
wants to merge 1 commit into from

Conversation

PraveenAdlakha
Copy link
Contributor

…on missing in workflow

@@ -87,7 +87,7 @@ private void onEnd(WorkflowExecutionContext context){
}
while(0<blockingQueue.remainingCapacity()){
try {
LOG.info("Sleeing, no capacity in threadpool....");
LOG.debug("Sleeping, no capacity in threadpool....");
TimeUnit.MILLISECONDS.sleep(500);
} catch (InterruptedException e) {
e.printStackTrace();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had missed this last review. Please use logger to log exception.

@pallavi-rao
Copy link

UT updates to OozieFeedWorkflowBuilderTest and OozieProcessWorkflowBuilderTest, please?

@sandeepSamudrala
Copy link
Contributor

@PraveenAdlakha : Can you rebase your patch once?

@pallavi-rao
Copy link

@PraveenAdlakha mentioned that 272 has an additional commit and hence this PR has fewer changes.

Will merge these changes.

@asfgit asfgit closed this in 35b10b4 Aug 25, 2016
PraveenAdlakha added a commit to PraveenAdlakha/falcon that referenced this pull request Aug 26, 2016
…on missing in workflow

Author: Praveen Adlakha <adlakha.praveen@gmail.com>

Reviewers: @pallavi-rao

Closes apache#274 from PraveenAdlakha/2125
PraveenAdlakha added a commit to PraveenAdlakha/falcon that referenced this pull request Aug 26, 2016
…on missing in workflow

Author: Praveen Adlakha <adlakha.praveen@gmail.com>

Reviewers: @pallavi-rao

Closes apache#274 from PraveenAdlakha/2125
pallavi-rao pushed a commit to pallavi-rao/falcon that referenced this pull request Feb 16, 2018
…on missing in workflow

Author: Praveen Adlakha <adlakha.praveen@gmail.com>

Reviewers: @pallavi-rao

Closes apache#274 from PraveenAdlakha/2125
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants