Skip to content

Conversation

@ahmedabu98
Copy link
Contributor

We should relax our type checking and default null array values to empty arrays. This is already done for TableRows [1] and is also the behavior of GoogleSQL [2]

[1]

if (bqValue == null) {
if (fieldDescriptor.isOptional() || allowMissingRequiredFields) {
return null;
} else if (fieldDescriptor.isRepeated()) {
return Collections.emptyList();

[2] https://cloud.google.com/bigquery/docs/reference/standard-sql/data-types#array_type

@ahmedabu98 ahmedabu98 merged commit 301286f into apache:master Oct 1, 2024
reeba212 pushed a commit to reeba212/beam that referenced this pull request Dec 4, 2024
@Abacn Abacn mentioned this pull request Nov 11, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants