GitHub user davidcavazos opened a pull request:
https://github.com/apache/beam/pull/3798
CoGroupByKey snippet with concrete example
R: @aaltay
R: @melap
Having a concrete common example for the docs, Python, and Java will allow
referencing the same example on every language. This is more consistent and
also allows tests to validate what is described in the docs.
This PR creates new tags `model_group_by_key_cogroupbykey_tuple_inputs` and
`model_group_by_key_cogroupbykey_tuple_outputs`, in `snippets_test.py`, to be
used as a concrete example for `CoGroupByKey` snippet.
It also slightly changes the output formatting to make more evident how to
use the results of the transform.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/davidcavazos/beam cgbk_snippet
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/beam/pull/3798.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #3798
----
commit a3fe833b6fee4f22a955a925dd6b4ed84cdc1470
Author: David Cavazos <[email protected]>
Date: 2017-08-31T19:24:46Z
Added concrete example for CoGroupByKey snippet
commit aaed1815c557eeb98b859dd179cc105b1fe5a69e
Author: David Cavazos <[email protected]>
Date: 2017-08-31T19:32:57Z
Made unique labels for model_co_group_by_key_tuple transforms
commit 817045d4743ec9751f5cd8e6520493198691b289
Author: David Cavazos <[email protected]>
Date: 2017-08-31T21:50:25Z
Fixed model_co_group_by_key_tuple output format
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
