[Portable BQ] default null list to empty list#33919
Conversation
|
assign set of reviewers |
|
Assigning reviewers. If you would like to opt out of this review, comment R: @m-trieu for label java. Available commands:
The PR bot will only process comments in the main thread (not review comments). |
|
Can we cherry-pick this one? @jrmccluskey |
|
@liferoad RC1 is already out for a vote, so to get this in 2.63.0 we'd need to either find a regression that needs fixing in that RC or deem this fix important enough to move to an RC2 anyway |
|
I don't think it warrants a new RC since this isn't a new regression introduced in 2.63.0. If there happens to be a new RC for a different reason, it'd be cool to include this, but up to @jrmccluskey's judgement |
|
I already have another PR cherry picked to the release branch in case we go to RC2, so feel free to open a cherry pick PR once this is merged |
|
Looks like we're heading to RC2 if you want to get this in @ahmedabu98 |
|
Thanks for the heads up. |
Fixes a problem where, in the DLQ, failed rows containing null values for list fields will fail to convert to Beam Rows.