My original thought for this change was that Crunch uses the class name Distinct. SQL also uses the keyword distinct.
Maybe the rule should be changed to adjectives or verbs depending on the context. Using a verb to describe this class really doesn't connote what the class does as succinctly as the adjective. On Mon, Oct 24, 2016 at 9:40 AM Neelesh Salian <nsal...@cloudera.com> wrote: > Hello, > > First of all, thank you to Daniel, Robert and Jesse for their review on > this: https://issues.apache.org/jira/browse/BEAM-239 > > A point that came up was using verbs explicitly for Transforms. > Here is the PR: https://github.com/apache/incubator-beam/pull/1164 > > Posting it to help understand if we have a consensus for it and if yes, we > could perhaps document it for future changes. > > Thank you. > > -- > Neelesh Srinivas Salian > Engineer >
