[
https://issues.apache.org/jira/browse/BEAM-9116?focusedWorklogId=372529&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-372529
]
ASF GitHub Bot logged work on BEAM-9116:
----------------------------------------
Author: ASF GitHub Bot
Created on: 15/Jan/20 19:45
Start Date: 15/Jan/20 19:45
Worklog Time Spent: 10m
Work Description: mxm commented on pull request #10580: [BEAM-9116] Limit
the number of past invocations stored in JobService
URL: https://github.com/apache/beam/pull/10580#discussion_r367071785
##########
File path:
runners/java-fn-execution/src/test/java/org/apache/beam/runners/fnexecution/jobsubmission/InMemoryJobServiceTest.java
##########
@@ -186,7 +193,59 @@ public void testJobSubmissionUsesJobInvokerAndIsSuccess()
throws Exception {
verify(invocation, times(1)).start();
}
+ @Test
+ public void testInvocationCleanup() {
+ final JobApi.GetJobsRequest getJobsRequest =
JobApi.GetJobsRequest.newBuilder().build();
+
+ final int maxRunningJobs = maxInvocationHistory + new Random().nextInt(50);
Review comment:
I agree, not really necessary.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 372529)
Time Spent: 1h 20m (was: 1h 10m)
> Limit the number of past invocations stored in the job service
> --------------------------------------------------------------
>
> Key: BEAM-9116
> URL: https://issues.apache.org/jira/browse/BEAM-9116
> Project: Beam
> Issue Type: Bug
> Components: jobserver
> Reporter: Maximilian Michels
> Assignee: Maximilian Michels
> Priority: Major
> Fix For: 2.19.0
>
> Time Spent: 1h 20m
> Remaining Estimate: 0h
>
> The {{InMemoryJobService}} stores an unbounded number of past job
> invocations. When this job server is long-running this can cause memory
> issues, as seen with our test setup for running Python tests.
> We should limit the number of past job invocations via a flag in the job
> server with a sensible default.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
