[
https://issues.apache.org/jira/browse/BEAM-2645?focusedWorklogId=382670&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-382670
]
ASF GitHub Bot logged work on BEAM-2645:
----------------------------------------
Author: ASF GitHub Bot
Created on: 06/Feb/20 01:41
Start Date: 06/Feb/20 01:41
Worklog Time Spent: 10m
Work Description: lukecwik commented on pull request #10770: [BEAM-2645]
Define the display data model type
URL: https://github.com/apache/beam/pull/10770#discussion_r375602370
##########
File path: model/pipeline/src/main/proto/beam_runner_api.proto
##########
@@ -1205,58 +1207,58 @@ message FunctionSpec {
bytes payload = 3;
}
-// TODO: transfer javadoc here
-message DisplayData {
-
- // (Required) The list of display data.
- repeated Item items = 1;
-
- // A complete identifier for a DisplayData.Item
- message Identifier {
-
- // (Required) The transform originating this display data.
- string transform_id = 1;
-
- // (Optional) The URN indicating the type of the originating transform,
- // if there is one.
- string transform_urn = 2;
-
- string key = 3;
+// A set of well known URNs describing display data.
+//
+// All descriptions must contain how the value should be classified and how it
+// is encoded. Note that some types are logical types which convey contextual
+// information about the pipeline in addition to an encoding while others only
+// specify the encoding itself.
+message StandardDisplayData {
+ enum DisplayData {
+ // A string label and value. Has a payload containing an encoded
+ // LabelledStringPayload.
+ LABELLED_STRING = 0 [(beam_urn) = "beam:display_data:labelled_string:v1"];
Review comment:
It doesn't matter for these constant fields since we don't expect to use the
enum as an actual field anywhere on a message and only use them for lookups. We
can always change it if we need to in the future and it won't impact execution
or existing pipelines either.
----------------------------------------------------------------
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: 382670)
Time Spent: 50m (was: 40m)
> Implement DisplayData translation to/from protos
> ------------------------------------------------
>
> Key: BEAM-2645
> URL: https://issues.apache.org/jira/browse/BEAM-2645
> Project: Beam
> Issue Type: New Feature
> Components: sdk-java-core
> Reporter: Kenneth Knowles
> Priority: Major
> Time Spent: 50m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
