[
https://issues.apache.org/jira/browse/BEAM-4791?focusedWorklogId=124518&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-124518
]
ASF GitHub Bot logged work on BEAM-4791:
----------------------------------------
Author: ASF GitHub Bot
Created on: 18/Jul/18 17:37
Start Date: 18/Jul/18 17:37
Worklog Time Spent: 10m
Work Description: lukecwik commented on a change in pull request #5957:
[BEAM-4791] Portable Flink runner integration test.
URL: https://github.com/apache/beam/pull/5957#discussion_r203467620
##########
File path:
runners/java-fn-execution/src/main/java/org/apache/beam/runners/fnexecution/control/DockerJobBundleFactory.java
##########
@@ -69,6 +70,21 @@
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
Review comment:
As a future thought: We should expect that the environment proto that we are
passing around is changed so that it has different types, one which is
`docker`, another one which is `inprocess` and then we can use a selector to
choose which environment factory to use within Flink/Spark/...
----------------------------------------------------------------
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: 124518)
Time Spent: 4h (was: 3h 50m)
> 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: 4h
> 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)
