[
https://issues.apache.org/jira/browse/BEAM-4791?focusedWorklogId=123842&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-123842
]
ASF GitHub Bot logged work on BEAM-4791:
----------------------------------------
Author: ASF GitHub Bot
Created on: 16/Jul/18 22:55
Start Date: 16/Jul/18 22:55
Worklog Time Spent: 10m
Work Description: tweise commented on a change in pull request #5957:
[BEAM-4791] Portable Flink runner integration test.
URL: https://github.com/apache/beam/pull/5957#discussion_r202849978
##########
File path:
runners/java-fn-execution/src/main/java/org/apache/beam/runners/fnexecution/control/DockerJobBundleFactory.java
##########
@@ -69,6 +69,14 @@
public class DockerJobBundleFactory implements JobBundleFactory {
private static final Logger LOG =
LoggerFactory.getLogger(DockerJobBundleFactory.class);
+ /** Factory that creates {@link JobBundleFactory} for the given {@link
JobInfo}. */
+ public interface JobBundleFactoryFactory {
+ DockerJobBundleFactory create(JobInfo jobInfo) throws Exception;
+ }
+ // TODO: a hacky way to override the factory for testing.
+ // Should be replaced with mechanism that let's users configure their own
factory
+ public static JobBundleFactoryFactory factory = (jobInfo) -> new
DockerJobBundleFactory(jobInfo);
Review comment:
It cannot be made final as it is used by the test. But in the future we
might use a service loader to find the factory.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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: 123842)
Time Spent: 2h 10m (was: 2h)
> Integration test for portable Flink runner basic batch/streaming execution
> --------------------------------------------------------------------------
>
> Key: BEAM-4791
> URL: https://issues.apache.org/jira/browse/BEAM-4791
> Project: Beam
> Issue Type: Task
> Components: runner-flink
> Reporter: Thomas Weise
> Assignee: Thomas Weise
> Priority: Major
> Labels: portability
> Time Spent: 2h 10m
> Remaining Estimate: 0h
>
> Run a simple pipeline in embedded mode to verify the basic workings of
> JobInvocation/translation/execution for both, batch and streaming. Although
> there will be some overlap with (future) ValidatesRunner, this test is meant
> to run as part of the unit tests.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
