-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BEAM-2937] Update Portable Combine URNs to new URNs. #5128
Conversation
retest this please |
1 similar comment
retest this please |
be88cf8
to
937a238
Compare
Run Python PreCommit |
1 similar comment
Run Python PreCommit |
R: @lukecwik |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix up the comment and I'll merge this.
@@ -245,6 +245,25 @@ message StandardPTransforms { | |||
COMBINE_PGBKCV = 0 [(beam_urn) = "beam:transform:combine_pgbkcv:v1"]; | |||
COMBINE_MERGE_ACCUMULATORS = 1 [(beam_urn) = "beam:transform:combine_merge_accumulators:v1"]; | |||
COMBINE_EXTRACT_OUTPUTS = 2 [(beam_urn) = "beam:transform:combine_extract_outputs:v1"]; | |||
|
|||
// Represents the lifted part of a Combine.perKey() operation. Caches and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better to refer to the name and point to the doc then the java code and a vague description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Adding combine URNs based on the portable combines doc: https://s.apache.org/beam-runner-api-combine-model Keeping the old URNs to avoid breaking existing code.
Changing combine URNs based on the portable combines doc:
https://s.apache.org/beam-runner-api-combine-model
Follow this checklist to help us incorporate your contribution quickly and easily:
[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue.mvn clean verify
to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.