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-4231] Runner utility for coder instantiation #5274
Conversation
R: @tgroh |
|
||
/** Helpers to construct coders for gRPC port reads and writes. */ | ||
public class WireCoders { | ||
/** Creates an SDK-side wire coder for a port read/write for the given PCollection. */ | ||
/** | ||
* Creates an SDK-side wire coder for a port read/write for the given PCollection. Returns a |
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.
"The returned coder will be a windowed value coder containing..."
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.
I've added a @return
clause that documents this.
Jenkins? friend, buddy? |
Is Jenkins down? That didn't seem to do anything. |
Yeah looks like
…On Fri, May 4, 2018 at 10:59 AM Ben Sidhom ***@***.***> wrote:
Is Jenkins down? That didn't seem to do anything.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5274 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAaPKiLpqthiR6cjxC3AS3wxHE-Gscfmks5tvJcYgaJpZM4Tx20o>
.
|
I also slipped in some doc comment fixes.
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../gradlew build
to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.