Hello,

The PTransform Style Guide is live
https://beam.apache.org/contribute/ptransform-style-guide/ - a natural next
step is to audit Beam libraries for compliance and file JIRAs for places
that need to be fixed. It'd be great to finish these cleanups before
declaring Beam stable API.

Please take a look and file JIRAs / post suggestions on this thread!

I think it'll also make a great source of easy and useful work for new
contributors.

Some things I remember off the top of my head:
- TextIO, KafkaIO use coders improperly - coders should not be used as a
general-purpose byte parsing mechanism.
- HadoopFileSource is not packaged as a PTransform
- Some connectors, e.g. KafkaIO, should use AutoValue for their parameter
builders, but don't
- A few connectors improperly use
- Some transforms expose their transform type as "Something.Bound" and
"Something.Unbound", e.g. TextIO.Read.Bound - such names are banned

I filed an umbrella JIRA https://issues.apache.org/jira/browse/BEAM-1353 about
making existing Beam transforms comply with the guide - let's crowdsource
this!

Thanks.

Reply via email to