You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by davidcavazos <gi...@git.apache.org> on 2017/08/31 22:03:41 UTC

[GitHub] beam pull request #3798: CoGroupByKey snippet with concrete example

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 <dc...@google.com>
Date:   2017-08-31T19:24:46Z

    Added concrete example for CoGroupByKey snippet

commit aaed1815c557eeb98b859dd179cc105b1fe5a69e
Author: David Cavazos <dc...@google.com>
Date:   2017-08-31T19:32:57Z

    Made unique labels for model_co_group_by_key_tuple transforms

commit 817045d4743ec9751f5cd8e6520493198691b289
Author: David Cavazos <dc...@google.com>
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] beam pull request #3798: CoGroupByKey snippet with concrete example

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/beam/pull/3798


---