Hello folks,

I wanted to share the news of merging of BEAM-239
<https://issues.apache.org/jira/browse/BEAM-239> via PullRequest : PR1164
<https://github.com/apache/incubator-beam/pull/1164>.

*On the Pull Request:*
1) Refactored the code for the occurrences of RemoveDuplicates /
RemovedDuplicatesand replacing with the word "Distinct"

2) Also changing "DedupExample" to DistinctExample (since DedupExampleTest
was already changed on this PR). Added BEAM-879
<https://issues.apache.org/jira/browse/BEAM-879> for that.

*There was a large discussion that ensued on the Dev list since the JIRA
involved the following:*

1) Adding a non-verb to the list of transforms although there are
exceptions.

2) The changing of the occurrences of RemoveDuplicates / RemovedDuplicates
in the code and replacing with the word "Distinct" is a
backwards-incompatible API change.

3) It's best to post a discussion for such changes on the dev@ list prior
to moving forward to help contributors and committers alike to chime in and
reach a conclusion.
A Vote would be a way to go for a large design change but, I think, when in
doubt, it's best to ask.

Feel free to add your thoughts.
Thank you.

Regards,

-- 
Neelesh Srinivas Salian
Engineer

Reply via email to