You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@slider.apache.org by go...@apache.org on 2017/02/02 06:32:19 UTC
incubator-slider git commit: SLIDER-1200
TestLongLivedProcess#testSetenv and
TestWorkflowForkedProcessService#testSetenv fail in certain scenarios
Repository: incubator-slider
Updated Branches:
refs/heads/develop 5a83421b2 -> 8850d062b
SLIDER-1200 TestLongLivedProcess#testSetenv and TestWorkflowForkedProcessService#testSetenv fail in certain scenarios
Project: http://git-wip-us.apache.org/repos/asf/incubator-slider/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-slider/commit/8850d062
Tree: http://git-wip-us.apache.org/repos/asf/incubator-slider/tree/8850d062
Diff: http://git-wip-us.apache.org/repos/asf/incubator-slider/diff/8850d062
Branch: refs/heads/develop
Commit: 8850d062b1dcf3de1608ced67a0860eeb340a975
Parents: 5a83421
Author: Gour Saha <go...@apache.org>
Authored: Wed Feb 1 22:31:59 2017 -0800
Committer: Gour Saha <go...@apache.org>
Committed: Wed Feb 1 22:31:59 2017 -0800
----------------------------------------------------------------------
.../slider/server/services/workflow/ForkedProcessService.java | 4 ++++
.../slider/server/services/workflow/TestLongLivedProcess.java | 2 ++
.../services/workflow/TestWorkflowForkedProcessService.java | 2 ++
3 files changed, 8 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/8850d062/slider-core/src/main/java/org/apache/slider/server/services/workflow/ForkedProcessService.java
----------------------------------------------------------------------
diff --git a/slider-core/src/main/java/org/apache/slider/server/services/workflow/ForkedProcessService.java b/slider-core/src/main/java/org/apache/slider/server/services/workflow/ForkedProcessService.java
index 352be49..9103321 100644
--- a/slider-core/src/main/java/org/apache/slider/server/services/workflow/ForkedProcessService.java
+++ b/slider-core/src/main/java/org/apache/slider/server/services/workflow/ForkedProcessService.java
@@ -152,6 +152,10 @@ public class ForkedProcessService
this.timeoutCode = code;
}
+ public void setRecentLineLimit(int limit) {
+ process.setRecentLineLimit(limit);
+ }
+
/**
* Build the process to execute when the service is started
* @param commandList list of commands is inserted on the front
http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/8850d062/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestLongLivedProcess.java
----------------------------------------------------------------------
diff --git a/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestLongLivedProcess.java b/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestLongLivedProcess.java
index 0eb2b78..7a7a327 100644
--- a/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestLongLivedProcess.java
+++ b/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestLongLivedProcess.java
@@ -136,6 +136,8 @@ public class TestLongLivedProcess extends WorkflowServiceTestBase implements
private LongLivedProcess initProcess(List<String> commands) {
process = new LongLivedProcess(name.getMethodName(), log, commands);
process.setLifecycleCallback(this);
+ // increase the line limit to accommodate containers with long list of envs
+ process.setRecentLineLimit(640);
return process;
}
http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/8850d062/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestWorkflowForkedProcessService.java
----------------------------------------------------------------------
diff --git a/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestWorkflowForkedProcessService.java b/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestWorkflowForkedProcessService.java
index bc1476a..71d6cbe 100644
--- a/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestWorkflowForkedProcessService.java
+++ b/slider-core/src/test/java/org/apache/slider/server/services/workflow/TestWorkflowForkedProcessService.java
@@ -133,6 +133,8 @@ public class TestWorkflowForkedProcessService extends WorkflowServiceTestBase {
process = new ForkedProcessService(name.getMethodName(), env,
commands);
process.init(new Configuration());
+ // increase the line limit to accommodate containers with long list of envs
+ process.setRecentLineLimit(640);
return process;
}