[
https://issues.apache.org/jira/browse/BEAM-651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15607333#comment-15607333
]
ASF GitHub Bot commented on BEAM-651:
-------------------------------------
GitHub user nssalian opened a pull request:
https://github.com/apache/incubator-beam/pull/1192
[BEAM-651]: Consider making TypedPValue.setTypeDescriptorInternal no longer
Internal
On the PR: Refactored the project to modify the name of
setTypeDescriptorInternal to setTypeDescriptor.
Ran a mvn clean install to SUCCESS and a local TRAVIS build went through as
well.
@kennknowles , do you have some to help review?
My previous PR had some additional conflicts.
Thank you.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nssalian/incubator-beam BEAM-651-new
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-beam/pull/1192.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 #1192
----
commit c8f48fb226e757019d05960225b744369f05ee42
Author: Neelesh Srinivas Salian <[email protected]>
Date: 2016-10-26T03:34:18Z
BEAM-651:Refactored the project to modify the name of
setTypeDescriptorInternal to setTypeDescriptor
----
> Consider making TypedPValue.setTypeDescriptorInternal no longer Internal
> ------------------------------------------------------------------------
>
> Key: BEAM-651
> URL: https://issues.apache.org/jira/browse/BEAM-651
> Project: Beam
> Issue Type: Wish
> Components: sdk-java-core
> Reporter: Kenneth Knowles
> Assignee: Neelesh Srinivas Salian
> Priority: Minor
> Labels: easy, easyfix, starter
>
> This would give fairly pithy answers to StackOverflow questions sometimes.
> When choosing between .getOutputCoder() and .getOutputTypeDescriptor() for a
> transform/DoFn we often choose the type, so the coder registry can do its
> thing.
> This would also give a similar choice between .setCoder(...) and
> .setTypeDescriptor(...).
> And anyhow we have the intention of removing our practice of the "*Internal"
> suffix, so this one might be most easily solved by making it public.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
