[
https://issues.apache.org/jira/browse/BEAM-7668?focusedWorklogId=273506&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-273506
]
ASF GitHub Bot logged work on BEAM-7668:
----------------------------------------
Author: ASF GitHub Bot
Created on: 08/Jul/19 20:06
Start Date: 08/Jul/19 20:06
Worklog Time Spent: 10m
Work Description: chadrik commented on pull request #8977: [BEAM-7668]
Add GetPipeline method to gRPC JobService
URL: https://github.com/apache/beam/pull/8977#discussion_r301277016
##########
File path:
runners/java-fn-execution/src/main/java/org/apache/beam/runners/fnexecution/jobsubmission/InMemoryJobService.java
##########
@@ -234,6 +237,26 @@ public void getState(
}
}
+ @Override
+ public void getPipeline(
+ GetJobPipelineRequest request, StreamObserver<GetJobPipelineResponse>
responseObserver) {
+ LOG.trace("{} {}", GetJobPipelineRequest.class.getSimpleName(), request);
+ String invocationId = request.getJobId();
+ try {
+ JobInvocation invocation = getInvocation(invocationId);
+ RunnerApi.Pipeline pipeline = invocation.getPipeline();
+ GetJobPipelineResponse response =
+ GetJobPipelineResponse.newBuilder().setPipeline(pipeline).build();
+ responseObserver.onNext(response);
+ responseObserver.onCompleted();
+ } catch (Exception e) {
+ String errMessage =
+ String.format("Encountered Unexpected Exception for Invocation %s",
invocationId);
+ LOG.error(errMessage, e);
Review comment:
Again, this pattern is repeated in 5 other places in this file. I was just
trying to stay consistent.
If you don't like the style used in the file, I would recommend addressing
them all at once in another PR.
----------------------------------------------------------------
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: 273506)
Time Spent: 1h 20m (was: 1h 10m)
> Add ability to query a pipeline from a gRPC JobService
> ------------------------------------------------------
>
> Key: BEAM-7668
> URL: https://issues.apache.org/jira/browse/BEAM-7668
> Project: Beam
> Issue Type: Improvement
> Components: beam-model
> Reporter: Chad Dombrova
> Assignee: Chad Dombrova
> Priority: Major
> Labels: portability
> Time Spent: 1h 20m
> Remaining Estimate: 0h
>
> As a developer I want to query the pipeline definition of jobs that have
> already been submitted to the Job Service, so that I can write a UI to view
> and monitor Beam pipelines via the portability model.
>
> PR is coming soon!
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
