[BEAM-697] TextIO withHeader() and withFooter() overwrite validation setting to false

• Type: Bug
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Affects Version/s: 0.2.0-incubating, 0.3.0-incubating
• Fix Version/s: 0.3.0-incubating
• Component/s: beam-model
• Assignee: Jeffrey Payne
• Reporter: Jeffrey Payne
• Created: 01/Oct/16 17:37
• Updated: 16/May/20 13:21
• Resolved: 18/Oct/16 00:04

Description

TextIO.Write.Bound.withHeader() and TextIO.Write.Bound.withFooter() both pass a literal value of false to the constructor's validate parameter for the returned Bound instance. This could, obviously, lead to unintended behavior wrt pipeline validation. Looks like a simple copy and paste error from TextIO.Write.Bound.withoutValidation()

I'm about to push another iteration for a PR for BEAM-55 and have included the trivial fix for this bug it in that set of changes. Creating this in case my PR is not accepted right away. Please close if this issue is wholly unnecessary. Thanks!

Issue Links

• links to: GitHub Pull Request #1111