beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kennknowles <...@git.apache.org>
Subject [GitHub] beam pull request #1946: [BEAM-115] Add proto definition for Runner API
Date Wed, 08 Feb 2017 06:14:05 GMT
GitHub user kennknowles opened a pull request:

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

    [BEAM-115] Add proto definition for Runner API

    Be sure to do all of the following to help us incorporate your contribution
    quickly and easily:
    
     - [x] Make sure the PR title is formatted like:
       `[BEAM-<Jira issue #>] Description of pull request`
     - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
           Travis-CI on your fork and ensure the whole test matrix passes).
     - [x] Replace `<Jira issue #>` in the title with the actual Jira issue
           number, if there is one.
     - [x] If this contribution is large, please file an Apache
           [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.txt).
    
    ---
    
    These are the protocol buffers definitions corresponding to the Avro schema proposed in
https://s.apache.org/beam-runner-api.
    
    Differences from the schema there:
    
     - Graph structure is decoupled from what data annotations the nodes
     - Adopted names from the Fn API's proto for things that overlap
     - Added explicit URNs and payloads for primitives
     - Added outline of state and timers
     - Added Environment URLs
    
    I would like to merge this and separately port the Fn API to use shared structures as
appropriate, since that may involve more extensive coding work.
    
    Unlike #662, this is _not_ a WIP just for discussion. This is intended for immediate use
as the serialization format for various pieces of the pipeline to unblock work on the Fn API.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kennknowles/beam pipeline-proto

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/beam/pull/1946.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 #1946
    
----
commit 7bdd2869053bd3af7afdf68e36e1d54f868419ae
Author: Kenneth Knowles <klk@google.com>
Date:   2017-02-07T23:25:32Z

    Add proto definition for Runner API

----


---
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.
---

Mime
View raw message