[
https://issues.apache.org/jira/browse/BEAM-11151?focusedWorklogId=507417&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-507417
]
ASF GitHub Bot logged work on BEAM-11151:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 04/Nov/20 01:06
Start Date: 04/Nov/20 01:06
Worklog Time Spent: 10m
Work Description: robertwb commented on a change in pull request #13214:
URL: https://github.com/apache/beam/pull/13214#discussion_r517041964
##########
File path: model/pipeline/src/main/proto/beam_runner_api.proto
##########
@@ -296,6 +296,24 @@ message StandardPTransforms {
//
// Payload: WindowingStrategy#window_fn FunctionSpec
MERGE_WINDOWS = 7 [(beam_urn) = "beam:transform:merge_windows:v1"];
+
+ // A transform that translates a given element to its human-readable
+ // representation.
+ //
+ // Input: KV<nonce, element>
+ // Output: KV<nonce, string>
+ //
+ // For each given element, the implementation should return the
+ // human-readable representation. When possible, the implementation should
+ // call a user-overridable method on the type. For example, Java should
Review comment:
s/should/could/
##########
File path: model/pipeline/src/main/proto/beam_runner_api.proto
##########
@@ -296,6 +296,24 @@ message StandardPTransforms {
//
// Payload: WindowingStrategy#window_fn FunctionSpec
MERGE_WINDOWS = 7 [(beam_urn) = "beam:transform:merge_windows:v1"];
+
+ // A transform that translates a given element to its human-readable
+ // representation.
+ //
+ // Input: KV<nonce, element>
+ // Output: KV<nonce, string>
+ //
+ // For each given element, the implementation should return the
+ // human-readable representation. When possible, the implementation should
+ // call a user-overridable method on the type. For example, Java should
+ // call `toString()`, Python should call `str()`, Golang should call
+ // `String()`. Users can override `toString`, `__str__`, and `String` for
+ // each language, respectively. The nonce is used by a runner to associate
Review comment:
I'd omit the detail about what users can override, this is inherent in
the previous sentence.
----------------------------------------------------------------
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: 507417)
Time Spent: 40m (was: 0.5h)
> Add ToString support
> --------------------
>
> Key: BEAM-11151
> URL: https://issues.apache.org/jira/browse/BEAM-11151
> Project: Beam
> Issue Type: New Feature
> Components: beam-model
> Reporter: Sam Rohde
> Assignee: Sam Rohde
> Priority: P2
> Time Spent: 40m
> Remaining Estimate: 0h
>
> This tracks work on adding the ToString to Beam. This new transform takes an
> element and returns a human-readable string back.
> design-doc:
> https://docs.google.com/document/d/1v7iWj0LIum04mYwRM_Cvze915tATwmEzLrqj_uVBkCE/edit#
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
