Skip to content

DRAFT: Protobuf schema review - #10215

Closed
reuvenlax wants to merge 2 commits into
apache:masterfrom
reuvenlax:protobuf_schema_review
Closed

DRAFT: Protobuf schema review#10215
reuvenlax wants to merge 2 commits into
apache:masterfrom
reuvenlax:protobuf_schema_review

Conversation

@reuvenlax

Copy link
Copy Markdown
Contributor

Please add a meaningful description for your change here


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Choose reviewer(s) and mention them in a comment (R: @username).
  • Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Post-Commit Tests Status (on master branch)

LangSDKApexDataflowFlinkGearpumpSamzaSpark
GoBuild Status------Build Status------Build Status
JavaBuild StatusBuild StatusBuild StatusBuild Status
Build Status
Build Status
Build StatusBuild StatusBuild Status
Build Status
Build Status
PythonBuild Status
Build Status
Build Status
Build Status
---Build Status
Build Status
Build Status
Build Status
------Build Status
XLang---------Build Status---------

Pre-Commit Tests Status (on master branch)

---JavaPythonGoWebsite
Non-portableBuild StatusBuild Status
Build Status
Build StatusBuild Status
Portable---Build Status------

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@reuvenlax
reuvenlaxforce-pushed the protobuf_schema_review branch 2 times, most recently from a44f082 to e838506CompareDecember 5, 2019 21:35
@reuvenlax
reuvenlaxforce-pushed the protobuf_schema_review branch from 3da325b to a6f369dCompareDecember 11, 2019 18:52
@reuvenlax
reuvenlaxforce-pushed the protobuf_schema_review branch from 6267882 to decc78bCompareDecember 21, 2019 18:36
@alexvanboxel

Copy link
Copy Markdown
Contributor

I think this PR can be closed as most of the changes are in. Only thing left to review is this:
#10502

Any objections closing this one?

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@reuvenlax@alexvanboxel