You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Stas Levin (JIRA)" <ji...@apache.org> on 2016/12/28 08:27:58 UTC
[jira] [Commented] (BEAM-849) Redesign PipelineResult API
[ https://issues.apache.org/jira/browse/BEAM-849?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15782396#comment-15782396 ]
Stas Levin commented on BEAM-849:
---------------------------------
In addition, the following points have been brought up in a dev list discussion:
1. The fact both {{waitUntilFinish()}} and {{cancel()}} return a {{State}} which may or may not be consistent with the {{State}} returned by (independently) calling {{getSate()}} is somewhat confusing. Perhaps it would make sense to have {{waitUntilFinish()}} return {{boolean}} to indicate timeouts while {{cancel()}} can be {{void}}. This way the only way to get the {{State}} would be via {{getSate}} which could help things keep consistent.
{code:java}
void getState();
boolean waitUntilFinish(...);
State cancel() throws IOException;
{code}
2. At the moment {{waitUntilFinish()}} returns {{null}} to indicate a timeout, which is a bit counter-intuitive for users in light of {{State}} being an enum (so they might not expect {{null}} to be ever returned).
> Redesign PipelineResult API
> ---------------------------
>
> Key: BEAM-849
> URL: https://issues.apache.org/jira/browse/BEAM-849
> Project: Beam
> Issue Type: Improvement
> Components: sdk-java-core
> Reporter: Pei He
> Assignee: Davor Bonaci
>
> Current state:
> Jira https://issues.apache.org/jira/browse/BEAM-443 addresses waitUntilFinish() and cancel().
> However, there are additional work around PipelineResult:
> need clearly defined contract and verification across all runners
> need to revisit how to handle metrics/aggregators
> need to be able to get logs
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)