TheNeuralBit commented on pull request #12035: URL: https://github.com/apache/beam/pull/12035#issuecomment-650394222
Thanks @pabloem, great suggestion. I made the documentation into javadoc, and also tried to make it more clear, PTAL I also changed the default to `ACCEPT_MISSING_OR_NULL` I think this is what most users will want, and its consistent with the ["postel principle"](https://en.wikipedia.org/wiki/Robustness_principle) (which I just learned about :P): "be conservative in what you do, be liberal in what you accept from others." Technically I think that is a breaking change since its possible people are relying on the old behavior, so I added an entry to that effect in CHANGES.md ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
