[BEAM-12532] BigQueryUtils fromBeamField omits ":00" from times with 0 seconds

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Affects Version/s: 2.30.0
• Fix Version/s: 2.32.0
• Component/s: io-java-gcp
• Assignee: Matthew Ouyang
• Reporter: Matthew Ouyang
• Created: 23/Jun/21 03:24
• Updated: 09/Jul/21 17:24
• Resolved: 09/Jul/21 17:24

Description

If a value in a Beam Row is of the SqlTypes.TIME data type, BigQueryUtils uses `toString()` to print out the time value [1]. `toString()` tries to optimize the string length by omitting nanoseconds if it is zero, and seconds if it is also zero [2]. However the BigQuery time type requires seconds to always be there even if it's zero [3].

[1] https://github.com/apache/beam/blob/58e2ea9c40fd08e24f23183852970f566b17d6cd/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/BigQueryUtils.java#L551
[2] https://docs.oracle.com/javase/8/docs/api/java/time/LocalTime.html#toString--
[3] https://cloud.google.com/bigquery/docs/reference/standard-sql/data-types#time_type

Issue Links

• links to: GitHub Pull Request #15070